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

Pin net6 branch #9051

Merged
merged 2 commits into from
Aug 1, 2022
Merged

Pin net6 branch #9051

merged 2 commits into from
Aug 1, 2022

Conversation

PureWeen
Copy link
Member

Description of Change

Pin NET6 branch top NET6 so when NET7 is installed we can still compile

@PureWeen PureWeen requested a review from mattleibow July 28, 2022 18:05
@PureWeen PureWeen changed the base branch from main to net6.0 July 28, 2022 18:38
@@ -5,5 +5,10 @@
"msbuild-sdks": {
"MSBuild.Sdk.Extras": "3.0.44",
"Microsoft.Build.NoTargets": "3.3.0"
},
"sdk": {
"version": "6.0.302",
Copy link
Member Author

Choose a reason for hiding this comment

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

If you set this to 6.0.400-preview CI fails because CI only has GA installed which is 6.0.302

Once 6.0.400 is GA then we can bump.

Copy link
Member

Choose a reason for hiding this comment

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

hum, how is this updated with new sdk bumps coming from maestro? we install local versions of dotnet and workloads,

Copy link
Member Author

Choose a reason for hiding this comment

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

It doesn't really matter because I set

"rollForward": "latestMinor"

So dotnet is going to resolve to whatever the newest pre 7 version is.

If you pull down this PR and run

dotnet --version

It'll return 6.0.400-previewXXXX

@Eilon Eilon added the area-infrastructure CI, Maestro / Coherency, upstream dependencies/versions label Jul 29, 2022
@PureWeen PureWeen merged commit 64e944f into net6.0 Aug 1, 2022
@PureWeen PureWeen deleted the set_global_json branch August 1, 2022 15:22
@github-actions github-actions bot locked and limited conversation to collaborators Dec 19, 2023
@samhouts samhouts added fixed-in-6.0.540 Look for this fix in 6.0.540 SR5! fixed-in-7.0.0-rc.1.6683 labels Aug 2, 2024
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
area-infrastructure CI, Maestro / Coherency, upstream dependencies/versions fixed-in-6.0.540 Look for this fix in 6.0.540 SR5! fixed-in-7.0.0-rc.1.6683
Projects
None yet
Development

Successfully merging this pull request may close these issues.

5 participants