-
Notifications
You must be signed in to change notification settings - Fork 76
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Create dotnet tool for many of the scripts/tools in this repo #1168
Comments
Instead of taking this PR (#906) we should move RIT functionality to dotnet-roslyn (#1168) as a set of subcommands which correspond to various scripts Feel free to suggest that |
It would be nice if we could actually tag the github PR with a linked to the merged insertion somehow (comment might be too noisy, maybe we can edit the description?) |
We should clean up the RIT as part of this (remove the powershell scripts, consolidate the flags where possible). For example /updateexistingpr and /overwritepr could conceivably be combined since 90% of the time we use both together |
You are talking about #1177, comment is good because it creates a notification that would alert the author or collaborators that it is available for verification. We should probably also look at linked issues and give them notice as well. vs code's github issue bot made this a very nice experience when I encountered it this week. |
RIT will no longer get updates once the roslyn-tools tool has these subcommands. We will help partner teams migrate their insertion pipelines to the new tool and archive RIT. |
We should empower Roslyn devs with the ability to perform more infra tasks and make it easier to get answers to common infra questions.
We can start by migrating several existing scripts/tools to run from a common tool.
Possible Subcommands
The text was updated successfully, but these errors were encountered: