Repository navigation
fix: make the memory-safety rules opt-in, fix SourceLink opt-out, and… - #51
Merged
Merged
Conversation
… refresh the settings docs - The C# 15 updated memory-safety rules (MemorySafetyRules) are off by default again: the updated model is an experimental compiler feature, so tier 2 is opt-in. Tier 1 and the PRCFG0001 validation are unchanged, and AllowUnsafeBlocks is still never set. - DisableSourceLink=true now opts out for real. Sdk.props re-assigned DisableSourceLink to false for every non-Roslyn project, clobbering a repository pre-import value, so the documented opt-out silently did nothing and the GitHub provider was always added. - SourceLink is documented end to end: a wiki page, a README section and skill coverage explain the provider for each host (Azure Repos, Azure DevOps Server, GitLab, Bitbucket), the SourceLinkAzureDevOpsServerGitHost item, the CPM PackageVersion requirement and the CI detection. Microsoft.SourceLink.AzureDevOps.Git, which does not exist, is gone. - Required settings and versions, how to set the target framework(s), and the compiler-visible (build_property.*) properties are now consistent across the README, the wiki and the skills. - A repository-local skill, purview-buildsdk-settings-docs-sync, is tracked under .agents/skills with a .gitignore carve-out; InstalledSkillsAreNotTracked still fails if a skill the CLI installed is committed. - SourceLinkTests and Dotnet11FeaturesTests pin the new behaviour.
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
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
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.
… refresh the settings docs