Skip to content

Conversation

@Forgind
Copy link
Contributor

@Forgind Forgind commented Apr 30, 2024

No description provided.

@ghost ghost added Area-Infrastructure untriaged Request triage from a team member labels Apr 30, 2024
@Forgind Forgind marked this pull request as ready for review May 23, 2024 16:45

var rollback = GetRollback(directory);

VM.WriteFile("C:\\SdkTesting\\global.json", @"{""sdk"":{""workloadVersion"":""versionToUpdateTo""}}".Replace("versionToUpdateTo", versionToUpdateTo)).Execute().Should().Pass();
Copy link
Member

Choose a reason for hiding this comment

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

nit: is there a way to use the variable inline rather than the clunky replace logic? Just a bit wordy.

Copy link
Contributor Author

Choose a reason for hiding this comment

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

String interpolation is a bit hampered when the string you're creating actually needs { and }. I can try to see if there's an escape sequence for that, though.

@Forgind Forgind merged commit ce1e06b into dotnet:release/8.0.4xx Jun 10, 2024
@Forgind Forgind deleted the add-vm-test branch June 10, 2024 16:19
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Area-Infrastructure untriaged Request triage from a team member

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants