Skip to content
This repository has been archived by the owner on Aug 7, 2023. It is now read-only.

make warnings from the validator available rather then ignoring them #62

Merged
merged 1 commit into from
Jun 30, 2016

Conversation

dirk-thomas
Copy link
Member

If e.g. xmllint fails to access a referenced xml-model file the warning was not visible to the user. If the validation fails the remaining output it always shown in a message.

@dirk-thomas dirk-thomas force-pushed the show_problems_with_validation branch 6 times, most recently from a448fc4 to 9c189d8 Compare June 29, 2016 05:55
type: 'Error'
text: output
filePath: textEditor.getPath()
range: [[0, 0], [0, 0]]
Copy link
Member

Choose a reason for hiding this comment

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

If you omit this range it is still a valid message, linter will show it as associated with the file but not give it a range within the file.

Copy link
Member Author

Choose a reason for hiding this comment

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

That's why I always keep my own PRs open for a day or two. You always have a good comment 😉 Thanks - updated in c9e5bf8

@dirk-thomas dirk-thomas force-pushed the show_problems_with_validation branch from 49fc466 to cb1fbe2 Compare June 29, 2016 18:37
@dirk-thomas dirk-thomas merged commit d911c49 into master Jun 30, 2016
@dirk-thomas dirk-thomas deleted the show_problems_with_validation branch June 30, 2016 15:49
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants