Skip to content

Include signing key by BinLogToSln#183

Merged
ericstj merged 1 commit into
dotnet:mainfrom
jjonescz:signing
May 13, 2025
Merged

Include signing key by BinLogToSln#183
ericstj merged 1 commit into
dotnet:mainfrom
jjonescz:signing

Conversation

@jjonescz

@jjonescz jjonescz commented Apr 29, 2025

Copy link
Copy Markdown
Member

This should fix errors like

error CS0281: Friend access was granted by 'Microsoft.CodeAnalysis, Version=42.42.42.42, Culture=neutral, PublicKeyToken=31bf3856ad364e35', but the public key of the output assembly ('') does not match that specified by the InternalsVisibleTo attribute in the granting assembly.

which break some references (i.e., some parts don't appear as links when they should be clickable).

When this is merged, new version of the BinLogToSln tool should be published (not sure how looks like the official build does this automatically), because it actually runs as part of other repos' official builds.

@jjonescz jjonescz mentioned this pull request Apr 30, 2025
@jjonescz

Copy link
Copy Markdown
Member Author

cc @ericstj

@jjonescz

Copy link
Copy Markdown
Member Author

@ericstj for a review here as well, thanks!

@ericstj ericstj left a comment

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

LGTM, sorry I missed the previous ping.

@ericstj ericstj merged commit 32e4db7 into dotnet:main May 13, 2025
@jjonescz jjonescz deleted the signing branch May 13, 2025 16:29
joperezr added a commit to joperezr/source-indexer that referenced this pull request May 15, 2026
The fork has diverged ~21 commits since the last upstream sync (PR dotnet#184,
2025-05-12). Blindly re-running update-source-browser.ps1 risks silently
dropping local features (dotnet#183 signing key, dotnet#192 dedup, dotnet#193 source-generated
files, dotnet#255 net10 retarget, dotnet#257 Linux binlog fix, plus Dependabot bumps).

- 02: prepend warning block to the 'Updating the vendored SourceBrowser'
  section listing the divergent PRs and recommending cherry-picks over a
  full re-sync.
- 00: cross-reference the warning from the overview bullet.

Co-authored-by: Copilot <223556219+Copilot@users.noreply.github.com>
Sign up for free to 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.

3 participants