-
Notifications
You must be signed in to change notification settings - Fork 84
Linting issues template #257
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
|
|
||
| __author__ = "{{ cookiecutter.full_name }}" | ||
| __email__ = "{{ cookiecutter.email }}" | ||
| __version__ = __version__ |
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.
Did does nothing?
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.
this is handled in #258
|
|
||
| To make sure you have a proper linter configuration you need to follow the steps below. | ||
|
|
||
| 1. Update `description` and `keywords` fields in [setup.cfg]({{ cookiecutter.repository }}/blob/main/setup.cfg). |
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.
How do I need to update it? Run prospector first
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.
updated the text
sverhoeven
left a comment
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.
Some more work needed
Co-authored-by: Stefan Verhoeven <stefan.verhoeven@gmail.com>
sverhoeven
left a comment
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.
Instructions are clear and minimal.
This PR introduces several lint errors that should be fixed, see inline suggestions
{{cookiecutter.project_name}}/{{cookiecutter.package_name}}/my_module.py
Outdated
Show resolved
Hide resolved
…_module.py Co-authored-by: Stefan Verhoeven <stefan.verhoeven@gmail.com>
sverhoeven
left a comment
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.
LGTM
Fixes: #238
Refs: #227 #228
This pull request adds an issue template which will be use to create an issue about linting fixes.