-
Notifications
You must be signed in to change notification settings - Fork 1.6k
validate.cpp: report error lines
#5559
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
Conversation
This comment was marked as resolved.
This comment was marked as resolved.
This comment was marked as resolved.
This comment was marked as resolved.
This comment was marked as resolved.
This comment was marked as resolved.
1 is the "default" / "don't care" value used below.
|
Thanks! 😻 This should make it easier than ever for contributors to find and fix their validation problems. I pushed a bunch of nitpicks and one enhancement: using the new strategy for disallowed characters. This helps ADO present all errors on the first page, instead of hiding the rest behind more details. |
|
I'm mirroring this to the MSVC-internal repo - please notify me if any further changes are pushed. |
|
Thanks for making it easier than ever to conform to the repo's formatting requirements! 😻 📈 🎉 |
Closes #2349