Skip to content
New issue

Have a question about this project? # for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “#”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? # to your account

Add comment to keep package versions in sync with O#-roslyn #8424

Merged
merged 2 commits into from
Mar 14, 2023

Conversation

allisonchou
Copy link
Contributor

Summary of the changes

@allisonchou allisonchou requested review from a team as code owners March 14, 2023 02:10
@@ -110,8 +110,12 @@
<MicrosoftVisualStudioTextDataPackageVersion>$(MicrosoftVisualStudioPackagesVersion)</MicrosoftVisualStudioTextDataPackageVersion>
<MicrosoftVisualStudioTextImplementationPackageVersion>$(MicrosoftVisualStudioPackagesVersion)</MicrosoftVisualStudioTextImplementationPackageVersion>
<MicrosoftVisualStudioTextLogicPackageVersion>$(MicrosoftVisualStudioPackagesVersion)</MicrosoftVisualStudioTextLogicPackageVersion>
<!-- NOTE: The Microsoft.VisualStudio.Threading package version should be kept in sync with O#-Roslyn's version:
https://github.com/OmniSharp/omnisharp-roslyn/blob/d7555ebfb6c4d7c6811c58370322d3b092c0abf6/build/Packages.props#L65 -->
Copy link
Member

Choose a reason for hiding this comment

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

Should there be a target branch linked here instead of the blob?

Copy link
Contributor Author

Choose a reason for hiding this comment

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

I think GitHub by default now copies line links via absolute commits. This might be so the intended linked line stays the same, as opposed to using a target branch where the line #s may change

eng/Versions.props Outdated Show resolved Hide resolved
@allisonchou allisonchou changed the title Add comment to keep package versions in sync in sync with O#-roslyn Add comment to keep package versions in sync with O#-roslyn Mar 14, 2023
@allisonchou allisonchou enabled auto-merge (squash) March 14, 2023 06:12
@allisonchou allisonchou merged commit 84fcdd9 into main Mar 14, 2023
@allisonchou allisonchou deleted the allichou/AddComment branch March 14, 2023 06:49
# for free to join this conversation on GitHub. Already have an account? # to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants