Skip to content

deps: Update roslyn package versions to 5.6.0 - #11047

Merged
vicancy merged 1 commit into
dotnet:mainfrom
filzrev:chore-update-roslyn-package-versions
Aug 11, 2026
Merged

deps: Update roslyn package versions to 5.6.0#11047
vicancy merged 1 commit into
dotnet:mainfrom
filzrev:chore-update-roslyn-package-versions

Conversation

@filzrev

@filzrevfilzrev commented Mar 25, 2026

Copy link
Copy Markdown
Collaborator

This PR update Roslyn and Microsoft.Build package to latest version.

Additionally, this PR include following PR content to pass CI tests
- #11039
- #11040

@filzrev
filzrev requested a review from yufeihMay 7, 2026 04:55
@filzrev
filzrevforce-pushed the chore-update-roslyn-package-versions branch 2 times, most recently from 4155ed8 to 0e16a2fCompareAugust 11, 2026 05:46
@filzrevfilzrev changed the title deps: Update roslyn package versions to 5.3.0deps: Update roslyn package versions to 5.6.0Aug 11, 2026
@filzrev
filzrevforce-pushed the chore-update-roslyn-package-versions branch from 0e16a2f to b3c38ccCompareAugust 11, 2026 05:47
@vicancy
vicancy merged commit 75bd9a1 into dotnet:mainAug 11, 2026
9 checks passed
MaceWindu added a commit to linq2db/docs that referenced this pull request Aug 13, 2026
* docs: qualify APIs by assembly instead of prefixing the namespace
The upstream branch reworked the UID prefix into a distinct assembly component, following review
feedback on dotnet/docfx#11090, so this switches to it:
linq2db.Tools.LinqToDB.Tools.Activity.ActivityStatistics ->
linq2db.Tools::LinqToDB.Tools.Activity.ActivityStatistics
`assemblyUidPrefixes` becomes `assemblyUids`, and since every prefix here was already the assembly's own
name, the map collapses to a plain list. `uidPrefixOverride` becomes `assemblyUidOverride` on the four
EntityFrameworkCore entries, which still need it because they all build `linq2db.EntityFrameworkCore`.
What this fixes on the site: the prefix used to be joined to the namespace with a dot and used as its
display name, so pages read as namespaces that do not exist. Now:
- `Namespace linq2db.Tools.LinqToDB.Tools.Activity` reads `Namespace LinqToDB.Tools.Activity
(linq2db.Tools)`.
- The namespace a type reports and the namespace page it links to agree; before, a type page said
`LinqToDB.Tools.Activity` while the page behind that link was titled with the made up name.
- Table of contents labels name the real namespace and the assembly it comes from, so the `LinqToDB`
declared by both `linq2db` and `linq2db.Tools` is now `LinqToDB (linq2db)` and `LinqToDB (linq2db.Tools)`
instead of two invented namespaces.
Page URLs change once more, as the file name carries `--` where the UID has `::`. Two hand written xrefs
in the articles named the old prefixed UIDs and are updated; they were the only ones.
docfx/ rebuilt from fix/8966-uid-prefixes (dd95c86d9). The custom Roslyn 5.6.0 commit is gone: upstream
took that bump in dotnet/docfx#11047, so the vendored build is now the upstream branch unmodified.
Build output: 0 errors, 2389 API files, 2407 HTML pages, and 6 docfx warnings, all pre-existing (5
`InvalidFileLink` from Microsoft docs relative links in linq2db's XML comments, 1 `InvalidBookmark` from a
`#remarks` anchor). No `DuplicateUids` and no `UidNotFound`.
Co-Authored-By: Claude Opus 5 (1M context) <noreply@anthropic.com>
* docs: refresh docfx build with the reviewed upstream branch
docfx/ rebuilt from fix/8966-uid-prefixes (9907932d6), which folds in a code review of the assembly
component design:
- a repeated or empty entry in the `assemblyUids` array is reported as an invalid entry instead of
throwing out of the JSON converter
- with `outputFormat: apiPage` or `markdown`, two assemblies resolving to one component now share a
single table of contents node
- the new table of contents ordering is skipped when no assembly is qualified, so it cannot affect a
project that does not use this
No change to `source/docfx.json` and no change to the output: 0 errors, 2389 API files, 2407 HTML pages,
and the same 6 pre-existing docfx warnings.
Co-Authored-By: Claude Opus 5 (1M context) <noreply@anthropic.com>
---------
Co-authored-by: Claude Opus 5 (1M context) <noreply@anthropic.com>
Sign up for freeto join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants

@filzrev@vicancy