Skip to content

Conversation

@smsaleem
Copy link

@smsaleem smsaleem commented Jul 22, 2025

I confirm that the pull-request:

  • Follows the contribution guidelines
  • Is based on my own work

@VolkmarR
Copy link
Contributor

It looks like you modified a generated file... If you look at the top of the file, it should show the source json file, that contains the definition used to generate it.

@smsaleem
Copy link
Author

smsaleem commented Jul 24, 2025

Means MSBuild.json needs an update manually ?

@VolkmarR
Copy link
Contributor

I'm non an expert in this, but i know, that there is a process to generate the tools definitions. First you need to update the msbuild.json and then run a nuke target to re-generate the files.

@valadas
Copy link
Contributor

valadas commented Jul 24, 2025

In think the target is GenerateTools, so you would run
nuke --target GenerateTools

But don't quote me on it, it has been a long time I had to do that...

@smsaleem
Copy link
Author

smsaleem commented Jul 25, 2025

Thanks. It worked.

@smsaleem smsaleem closed this Jul 25, 2025
@smsaleem
Copy link
Author

A separate PR for MSBuild.json has been created

@smsaleem smsaleem deleted the patch-1 branch August 4, 2025 16:56
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