-
Notifications
You must be signed in to change notification settings - Fork 34
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 ability to have custom lint checks #573
Conversation
58711ed
to
8cd09ae
Compare
8cd09ae
to
e64e5f0
Compare
@@ -0,0 +1 @@ | |||
#!/bin/bash |
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.
That's not the way it will work. This file will be wiped all the time by the plugin template.
@@ -71,6 +71,11 @@ jobs: | |||
run: flake8 --config flake8.cfg | |||
{%- endif %} | |||
|
|||
{% if extra_linting -%} | |||
- name: Run extra lint checks | |||
run: sh .ci/scripts/extra_linting.sh |
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 about something like [ -x .ci/scripts/extra_linting.sh] && .ci/scripts/extra_linting.sh
?
e64e5f0
to
ec914d7
Compare
d83a09a
to
db7a3df
Compare
CHANGES/572.misc
Outdated
@@ -0,0 +1 @@ | |||
Add ability to perform custom lint checks |
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.
Add ability to perform custom lint checks | |
Added the ability to perform custom lint checks. |
@@ -71,6 +71,11 @@ jobs: | |||
run: flake8 --config flake8.cfg | |||
{%- endif %} | |||
|
|||
{% if extra_linting -%} | |||
- name: Run extra lint checks | |||
run: [ -x .ci/scripts/extra_linting.sh] && .ci/scripts/extra_linting.sh |
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.
run: [ -x .ci/scripts/extra_linting.sh] && .ci/scripts/extra_linting.sh | |
run: [ -x .ci/scripts/extra_linting.sh ] && .ci/scripts/extra_linting.sh |
plugin-template
Outdated
@@ -22,6 +22,7 @@ DEFAULT_SETTINGS = { | |||
"upgrade_range": [], | |||
"black": True, | |||
"flake8": True, | |||
"extra_linting": False, |
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 don't think we need the double guard here.
db7a3df
to
2e8d559
Compare
{% if extra_linting -%} | ||
- name: Run extra lint checks | ||
run: [ -x .ci/scripts/extra_linting.sh ] && .ci/scripts/extra_linting.sh | ||
{%- endif %} |
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.
Sorry, forgot to mention: You need to remove the ifs and the docs for this flag again.
The whole PR would just be lines 75-76 (and the changelog).
2e8d559
to
963dc43
Compare
closes #572