-
Notifications
You must be signed in to change notification settings - Fork 178
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
Makman2/linterups #68
Conversation
@@ -205,6 +205,10 @@ We can test our bear like this | |||
by their **filename** and display them based on their | |||
**classname**. | |||
|
|||
Normally, providing a severity-map is not needed, as *coala* has a default |
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.
nooo no more italic on coala, i have taken thoughts and we dont want that :D
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.
haha alright :D
e7a3979
to
c0336f8
Compare
ack c0336f8 |
@@ -71,6 +71,17 @@ You can use these parameters to construct the command line arguments. The | |||
linter expects from you to return an argument sequence here. A tuple is | |||
preferred. We will do this soon for ``PylintTutorialBear``. | |||
|
|||
.. note:: | |||
|
|||
``create_arguments`` can also be a non-staticmethod. In this case you also |
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.
isnt it non-static method as in two words?
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.
I wanted to refer to the @staticmethod
decorator. Though yes you are right, this is no good english :3
I'll rephrase to
create_arguments
doesn't have to be a static method.
ack f1aa603 |
"value=2 = 2") | ||
return config_file | ||
|
||
The string returned from this method is written into a temporary file before |
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.
"returned by"
ack 88260f0 |
ack a7c0789 |
Default severities are written down at the API documentation and also maintained there.
This was documentation on the deprecated API.
It adds absolutely no information at all.
"that" is always restrictive in use, that's why it never takes a comma before it. Also use "which" in one place, as the sentence indicates nonrestrictive information.
Mention non-staticmethod'ness for `create_arguments()`.
c0336f8
to
a68d618
Compare
ack d2a8df2 |
@rultor merge |
No description provided.