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

Fix CI version in main #1273

Merged
merged 6 commits into from
Jun 9, 2023

Conversation

martincostello
Copy link
Member

  • Use v8alpha tag for pre-release builds in main.
  • Use the default file name for GitVersion configuration.
  • Include the build version in the assembly file version so we can tell build-to-build.
  • Remove logging of the informational version that isn't what we use.
  • Add .vscode/settings.json file the new C# Dev Kit keeps creating.
  • Use Minimal MSBuild verbosity for dotnet clean to match other targets.

Resolves #1270.

Use Minimal MSBuild verbosity for `dotnet clean` to match other targets.
- Use `v8alpha` tag for pre-release builds in `main`.
- Use the default file name for GitVersion configuration.
- Include the build version in the assembly file version.
- Remove logging of the informational version that isn't what we use.
The new C# Dev Kit keeps creating this file in the repo, so checking it in.
@martincostello martincostello added CI/build v8 Issues related to the new version 8 of the Polly library. labels Jun 9, 2023
@martincostello martincostello added this to the v8.0.0 milestone Jun 9, 2023
Use `CommitsSinceVersionSource` instead of `BuildMetaData` as that doesn't seem to be set in PRs.
@codecov
Copy link

codecov bot commented Jun 9, 2023

Codecov Report

Merging #1273 (da54104) into main (a908391) will not change coverage.
The diff coverage is n/a.

@@           Coverage Diff           @@
##             main    #1273   +/-   ##
=======================================
  Coverage   83.33%   83.33%           
=======================================
  Files         270      270           
  Lines        6317     6317           
  Branches      981      981           
=======================================
  Hits         5264     5264           
  Misses        844      844           
  Partials      209      209           
Flag Coverage Δ
linux 83.33% <ø> (ø)
macos 83.33% <ø> (ø)
windows 83.33% <ø> (ø)

Flags with carried forward coverage won't be shown. Click here to find out more.

next-version: 8.0.0
branches:
main:
tag: v8alpha
Copy link
Member Author

Choose a reason for hiding this comment

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

I've updated this based on our conversation yesterday, though the v8 prefix seems redundant to me.

@martincostello
Copy link
Member Author

Hmm - the package version is still 8.0.0...

Fix missing pre-release suffix for the NuGet packages.
We don't have packages with and without strong names anymore, so remove the word "Signed" to prevent confusion with Authenticode signing.
@martincostello martincostello marked this pull request as ready for review June 9, 2023 12:58
@martincostello martincostello merged commit 86ce2b1 into App-vNext:main Jun 9, 2023
@martincostello martincostello deleted the issue-1270-ci-version branch June 9, 2023 12:58
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
CI/build v8 Issues related to the new version 8 of the Polly library.
Projects
No open projects
Status: Done
Development

Successfully merging this pull request may close these issues.

Fix CI version in main to not use stable versions if not a tag
1 participant