-
Notifications
You must be signed in to change notification settings - Fork 221
Enable source-build pre-built detection #8492
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
Merged
Changes from all commits
Commits
Show all changes
14 commits
Select commit
Hold shift + click to select a range
fdd11bd
Enable source-build pre-built detection
NikolaMilosavljevic f765403
Update comment
NikolaMilosavljevic 049cc5d
Update dependency and automatic version updates
NikolaMilosavljevic a49bace
Update eng/SourceBuildPrebuiltBaseline.xml
NikolaMilosavljevic 702da69
Allow 2 roslyn versions while pinning 2 packages used in source-build
NikolaMilosavljevic 66c7617
Update comment in SourceBuildPrebuiltBaseline.xml
NikolaMilosavljevic 5d51c69
Merge branch 'main' into prebuilts
NikolaMilosavljevic 647f22f
Add explicit package dependencies
NikolaMilosavljevic 1f8ba5e
Address code-check issues
NikolaMilosavljevic 4e9c370
Merge branch 'main' into prebuilts
NikolaMilosavljevic f3e6e24
Update dependencies
NikolaMilosavljevic 5ecabe1
Update package versions
NikolaMilosavljevic f898e50
Merge branch 'main' into prebuilts
NikolaMilosavljevic 7ab376a
Fix coherency versions
NikolaMilosavljevic 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
| Original file line number | Diff line number | Diff line change |
|---|---|---|
| @@ -1,5 +1,11 @@ | ||
| <UsageData> | ||
| <IgnorePatterns> | ||
| <UsagePattern IdentityGlob="*/*" /> | ||
| <UsagePattern IdentityGlob="Microsoft.SourceBuild.Intermediate.*/*" /> | ||
|
|
||
| <!-- Ignoring the following prebuilts. net7.0 targeting is still needed | ||
| due to https://github.com/dotnet/razor/issues/8293#issuecomment-1454042002 | ||
| net7.0 targeting removal is tracked in https://github.com/dotnet/razor/issues/8531 --> | ||
| <UsagePattern IdentityGlob="Microsoft.AspNetCore.App.Ref/7.0.2" /> | ||
| <UsagePattern IdentityGlob="Microsoft.NETCore.App.Ref/7.0.2" /> | ||
| </IgnorePatterns> | ||
| </UsageData> | ||
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.
The reason for the two 7.0 targetting packs should be tracked with #8293. We do not want these to be solved with SBRP.
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.
Updated with 66c7617
Added a reference to reasons for
net7.0targeting and acreference to new tracking issue fornet7.0targeting removal.