[build] Remove omnisharp.json generation from xaprepare#11631
Open
jonathanpeppers wants to merge 1 commit into
Open
[build] Remove omnisharp.json generation from xaprepare#11631jonathanpeppers wants to merge 1 commit into
jonathanpeppers wants to merge 1 commit into
Conversation
The Get_Omnisharp_Json step in Step_GenerateFiles substituted two placeholders into build-tools/scripts/omnisharp.json.in and wrote an omnisharp.json file at the repo root for OmniSharp / VS Code C# extension users. The generated file was always .gitignored and nothing in the repo, build, or CI consumes it -- it was purely a local-dev convenience. Anyone who actually wants an omnisharp.json can keep their own local copy (it remains gitignored). Removing the generation slice is another small step in the slow xaprepare reduction, following the same pattern as PRs #11568, #11580, #11608, and #11613. Audit before / after: ``git grep -in omnisharp`` previously returned five hits and now returns only one -- the unrelated MSBuild context guard in xaprepare.csproj that prevents xaprepare''s targets from importing under OmniSharp''s MSBuild process. That guard is intentional and is left untouched. Changes: - Step_GenerateFiles.cs: drop the Get_Omnisharp_Json (context) entry from the generated-files list and delete the Get_Omnisharp_Json method. - Delete build-tools/scripts/omnisharp.json.in. - Remove /omnisharp.json from .gitignore (the file is no longer produced). The shared properties used by the deleted method (MicrosoftDotnetSdkInternalPackageVersion, DotNetPreviewPath) are still referenced by Step_InstallDotNetPreview and xaprepare.targets, so they remain. Co-authored-by: Copilot <223556219+Copilot@users.noreply.github.com>
Contributor
There was a problem hiding this comment.
Pull request overview
Removes local-developer-only omnisharp.json generation from xaprepare, continuing the ongoing reduction of xaprepare responsibilities in the build tooling.
Changes:
- Removed
Get_Omnisharp_JsonfromStep_GenerateFilesand deleted its implementation. - Deleted the
build-tools/scripts/omnisharp.json.intemplate. - Updated
.gitignoreto stop ignoring/omnisharp.json.
Reviewed changes
Copilot reviewed 2 out of 3 changed files in this pull request and generated no comments.
| File | Description |
|---|---|
| build-tools/xaprepare/xaprepare/Steps/Step_GenerateFiles.cs | Removes the omnisharp-generated-file entry and its generator method. |
| build-tools/scripts/omnisharp.json.in | Deletes the unused OmniSharp configuration template. |
| .gitignore | Removes the repo-root omnisharp.json ignore entry. |
jonathanpeppers
added
the
ready-to-review
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.
Context
Get_Omnisharp_JsoninStep_GenerateFilessubstituted two placeholders intobuild-tools/scripts/omnisharp.json.inand wroteomnisharp.jsonat the repo root for OmniSharp / VS Code C# extension users. The generated file was always.gitignored and nothing in the repo, build, or CI consumes it — it was purely a local-dev convenience.Anyone who actually wants an
omnisharp.jsoncan keep their own local copy (it remains gitignored). Removing this generation slice is another small step in the slow xaprepare reduction, continuing the pattern from #11568, #11580, #11608, and #11613.Audit
Before this change
git grep -in "omnisharp"returned five hits. After it returns only one — the unrelated MSBuild-context guard inxaprepare.csprojthat prevents xaprepare's targets from importing under OmniSharp's MSBuild process:That guard is intentional and is not touched by this PR.
Changes
build-tools/xaprepare/xaprepare/Steps/Step_GenerateFiles.cs: drop theGet_Omnisharp_Json (context)entry from the generated-files list and delete theGet_Omnisharp_Jsonmethod.build-tools/scripts/omnisharp.json.in./omnisharp.jsonfrom.gitignore— the file is no longer produced.The shared properties used by the deleted method (
MicrosoftDotnetSdkInternalPackageVersion,DotNetPreviewPath) are still referenced byStep_InstallDotNetPreviewandxaprepare.targets, so they remain.Verification
dotnet build build-tools/xaprepare/xaprepare/xaprepare.csproj -c Debug— 0 warnings, 0 errors.git grep -in "omnisharp"— returns only the unrelatedxaprepare.csprojguard described above.Documentation/references to omnisharp; no.vscode/folder in the repo.