Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Add SuppressFinalPackageVersion to libraries we don't plan to ship stable yet #3668

Merged
merged 1 commit into from
Apr 15, 2024

Conversation

eerhardt
Copy link
Member

@eerhardt eerhardt commented Apr 12, 2024

  • Aspire.Hosting.AWS
  • OpenAI
  • EventHubs

Part of #2541

FYI - @normj

Microsoft Reviewers: Open in CodeFlow

…able yet

- Aspire.Hosting.AWS
- OpenAI
- EventHubs

Part of dotnet#2541
@dotnet-issue-labeler dotnet-issue-labeler bot added the area-integrations Issues pertaining to Aspire Integrations packages label Apr 12, 2024
@@ -3,6 +3,8 @@
<PropertyGroup>
<TargetFramework>$(NetCurrent)</TargetFramework>
<IsPackable>true</IsPackable>
<!-- This library needs more bake time until it can ship stable. -->
<SuppressFinalPackageVersion>true</SuppressFinalPackageVersion>
Copy link
Member

Choose a reason for hiding this comment

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

Just for my own understanding as this is the first time I have seen this one

https://github.com/dotnet/arcade/blob/main/Documentation/CorePackages/Versioning.md?plain=1#L119-L120

@mitchdenny
Copy link
Member

Why not Open AI?

Copy link
Member

@joperezr joperezr left a comment

Choose a reason for hiding this comment

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

Thanks Eric!

@eerhardt
Copy link
Member Author

Why not Open AI?

The underlying client library isn't stable. https://www.nuget.org/packages/Azure.AI.OpenAI/

@eerhardt
Copy link
Member Author

Merging. If there are more packages that should stay pre-release we can add them later.

@eerhardt eerhardt merged commit 62f6570 into dotnet:main Apr 15, 2024
8 checks passed
@eerhardt eerhardt deleted the PrereleasePackages branch April 15, 2024 16:44
@eerhardt
Copy link
Member Author

/backport to release/8.0

Copy link
Contributor

Started backporting to release/8.0: https://github.com/dotnet/aspire/actions/runs/8692901723

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
area-integrations Issues pertaining to Aspire Integrations packages
Projects
None yet
Development

Successfully merging this pull request may close these issues.

5 participants