Skip to content
This repository was archived by the owner on Jan 13, 2025. It is now read-only.

Conversation

@jpobst
Copy link
Contributor

@jpobst jpobst commented Jun 14, 2022

Context: xamarin/GooglePlayServicesComponents#617
Context: xamarin/GooglePlayServicesComponents#621
Context: xamarin/GooglePlayServicesComponents#624

GPS binds 2 pre-release packages:

  • Xamarin.Google.Android.ODML.Image 1.0.0-beta1
  • Xamarin.GooglePlayServices.MLKit.LanguageId 117.0.0.1-beta1

Attempting to consume these results in XBD errors because the pre-release specifier is rejected:

error XBD020: Invalid item ID image-1.0.0-beta1

We need to expand our version regex to support pre-release specifiers so these can be downloaded.

@jpobst jpobst force-pushed the xbd-pre-release branch 3 times, most recently from df0518e to 84d405d Compare June 16, 2022 14:21
@jpobst jpobst force-pushed the xbd-pre-release branch from 84d405d to d411aee Compare June 16, 2022 14:22
@jpobst jpobst marked this pull request as ready for review June 16, 2022 14:34
@jpobst jpobst requested review from Redth, mattleibow and moljac June 16, 2022 14:34
@samhouts samhouts added the partner/cat 😻 Client CAT Team label Jun 21, 2022
@AdamEssenmacher
Copy link

NuGet semver docs, for reference: https://docs.microsoft.com/en-us/nuget/concepts/package-versioning

@moljac
Copy link
Contributor

moljac commented Jul 28, 2022

/azp run

@azure-pipelines
Copy link

Azure Pipelines successfully started running 1 pipeline(s).

@moljac
Copy link
Contributor

moljac commented Jul 28, 2022

/azp run

@azure-pipelines
Copy link

Azure Pipelines successfully started running 1 pipeline(s).

Copy link
Member

@jonathanpeppers jonathanpeppers left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM


<PropertyGroup>
<TargetFramework>netcoreapp3.0</TargetFramework>
<TargetFramework>net6</TargetFramework>
Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Hmm, didn't realize this worked, most people put:

Suggested change
<TargetFramework>net6</TargetFramework>
<TargetFramework>net6.0</TargetFramework>

Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

net6 works, but you were right net6.0 is preferred.

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Labels

partner/cat 😻 Client CAT Team

Projects

None yet

Development

Successfully merging this pull request may close these issues.

6 participants