Skip to content
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

MSBuild ValidateTemplates task PoC #5840

Merged

Conversation

vlada-shubina
Copy link
Member

@vlada-shubina vlada-shubina commented Dec 28, 2022

Problem

Template validation

Solution

follow up for #5838
usage of new validation from MSBuild authoring tasks

TODO:

  • extract localization strings (once the approach is approved)
  • better location reporting (separate PR)
  • info messages are not reported in MS in Error list

Checks:

  • Added unit tests
  • Added #nullable enable to all the modified files ?

@vlada-shubina vlada-shubina requested a review from a team as a code owner December 28, 2022 14:45
@YuliiaKovalova
Copy link
Member

Looks good!

Copy link
Member

@JanKrivanek JanKrivanek left a comment

Choose a reason for hiding this comment

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

Looks solid!

I added couple minor things for consideration.
The main thing is localization - not sure if the task has access to resources when loaded by MSBuild though

@vlada-shubina vlada-shubina changed the base branch from dev/vshubina/validation-poc to feature/validation March 14, 2023 16:11
@vlada-shubina vlada-shubina merged commit 9cc1b2c into dotnet:feature/validation Mar 15, 2023
vlada-shubina added a commit that referenced this pull request Mar 15, 2023
* MSBuild validate task PoC

* review comments

* localization
vlada-shubina added a commit that referenced this pull request Mar 17, 2023
* MSBuild validate task PoC

* review comments

* localization
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