-
-
Notifications
You must be signed in to change notification settings - Fork 8
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
Add basic manifest check. #113
Conversation
@@ -39,6 +40,7 @@ | |||
PreferSingleQuotes, | |||
UTF8Headers, | |||
ZPTLint, | |||
CheckManifest, |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Can we keep them sorted alphabetically :)
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Yes, this also defines the order in which they are run.
what about adding some tests? |
@hvelarde can you please read again what i wrote in the PR description? |
@saily morning blindness, sorry... |
e47d682
to
60696fe
Compare
This fixes #69, if i look into the code, the discussion was pretty long, but the code fragment is really compact.
A test is missing, so i marked this PR as [WIP], but just to let you know i'm working on it.