-
Notifications
You must be signed in to change notification settings - Fork 544
[main] Update dependencies from dotnet/dotnet #10073
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
Merged
jonathanpeppers
merged 8 commits into
main
from
darc-main-3eee70c5-7f9a-41a5-8224-e3c14aae17e6
Apr 25, 2025
Merged
Changes from all commits
Commits
Show all changes
8 commits
Select commit
Hold shift + click to select a range
4722433
Update dependencies from https://github.com/dotnet/dotnet build 20250…
dotnet-maestro[bot] 61d94d6
Remove CoherentParentDependency, it's no longer needed
akoeplinger ed257eb
Update Versions.props
akoeplinger c10a39e
Add back Microsoft.DotNet.ApiCompat
akoeplinger f321a77
Fix versions
akoeplinger 3b60c22
Fix versions
akoeplinger 031e5d9
dotnet/runtime 8.0.16 feeds
jonathanpeppers d3bac29
dotnet/runtime 9.0.5 feeds
jonathanpeppers File filter
Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
There are no files selected for viewing
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
@pjcollins is there any concern updating these? Or do some of our yaml templates use a different dotnet/arcade version anyway?
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
We can add this and the arcade dependency above to the excluded asset list, if you don't want them to get updated from the VMR, let us know please
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
@dkurepa do you know if there a way to ignore certain dependencies in a subscription? This would help us update the downlevel .NET 8/9 packs on main more effectively.
For example, I was trying to track
Microsoft.NET.Workload.Mono.ToolChain.Current.Manifest-8.0.100
here from dotnet/runtime/release/8.0:android/eng/Version.Details.xml
Lines 27 to 31 in 1d79963
But I bet it will update on the next dotnet/runtime/release/9.0 update.
I think this PR is probably fine as-is, I just wanted to check about the two dependencies before merging.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
yes! The
--excluded-assets
flag should help you with this. So you can just rundarc update-subscription --id 58e8279d-a615-4b62-adba-499b6c5c9d3e --excluded-assets <list of assets separated by ;>
. Note that this command will always update the whole list of excluded assets, so always list all of themThere was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
For the subscription:
I tried:
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I don't think this will be an issue for us, and it's probably for the best for these to be updated more regularly. Our MSI generation tasks still are pinned on a specific arcade version in a separate repo.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
@jonathanpeppers tracking
Microsoft.NET.Workload.Mono.ToolChain.Current.Manifest-8.0.100
makes no sense in this case since that is produced in a completely different channel.Let's talk about the use case you have for this separately, you can ping me on Teams