Skip to content
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

[#664] pylint upgrade #665

Merged
merged 2 commits into from
Dec 21, 2018
Merged
Show file tree
Hide file tree
Changes from all commits
Commits
File filter

Filter by extension

Filter by extension


Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
55 changes: 25 additions & 30 deletions Jenkinsfile
Original file line number Diff line number Diff line change
Expand Up @@ -195,42 +195,37 @@ pipeline {
}
steps {
sh '''
cd legion
pycodestyle --show-source --show-pep8 legion
pycodestyle --show-source --show-pep8 tests --ignore E402,E126,W503,E731
pydocstyle --source legion
pycodestyle --show-source --show-pep8 legion/legion
pycodestyle --show-source --show-pep8 legion/tests --ignore E402,E126,W503,E731
pydocstyle --source legion/legion

TERM="linux" pylint --persistent=n legion > legion-pylint.log || exit 0
TERM="linux" pylint --persistent=n tests >> legion-pylint.log || exit 0
cd ..
'''

archiveArtifacts 'legion/legion-pylint.log'
warnings canComputeNew: false, canResolveRelativePaths: false, categoriesPattern: '', defaultEncoding: '', excludePattern: '', healthy: '', includePattern: '', messagesPattern: '', parserConfigurations: [[ parserName: 'PyLint', pattern: 'legion/legion-pylint.log']], unHealthy: ''
pycodestyle --show-source --show-pep8 legion_airflow/legion_airflow
pycodestyle --show-source --show-pep8 legion_airflow/tests
pydocstyle legion_airflow/legion_airflow

sh '''
cd legion_airflow
pycodestyle --show-source --show-pep8 legion_airflow
pycodestyle --show-source --show-pep8 tests
pydocstyle legion_airflow
# Because of https://github.com/PyCQA/pylint/issues/352 or need to fix PYTHONPATH in unit tests
touch legion/tests/__init__.py

TERM="linux" pylint --persistent=n legion_airflow > legion_airflow-pylint.log || exit 0
TERM="linux" pylint --persistent=n tests >> legion_airflow-pylint.log || exit 0
cd ..
'''

archiveArtifacts 'legion_airflow/legion_airflow-pylint.log'
warnings canComputeNew: false, canResolveRelativePaths: false, categoriesPattern: '', defaultEncoding: '', excludePattern: '', healthy: '', includePattern: '', messagesPattern: '', parserConfigurations: [[ parserName: 'PyLint', pattern: 'legion_airflow/legion_airflow-pylint.log']], unHealthy: ''

sh '''
cd legion_test
TERM="linux" pylint --exit-zero --output-format=parseable --reports=no legion/legion > legion-pylint.log
TERM="linux" pylint --exit-zero --output-format=parseable --reports=no legion/tests >> legion-pylint.log

TERM="linux" pylint --persistent=n legion_test > legion_test-pylint.log || exit 0
cd ..
TERM="linux" pylint --exit-zero --output-format=parseable --reports=no legion_airflow/legion_airflow >> legion-pylint.log
TERM="linux" pylint --exit-zero --output-format=parseable --reports=no legion_airflow/tests >> legion-pylint.log
TERM="linux" pylint --exit-zero --output-format=parseable --reports=no legion_test/legion_test >> legion-pylint.log
# Because of https://github.com/PyCQA/pylint/issues/352 or need to fix PYTHONPATH in unit tests
rm -rf legion/tests/__init__.py
'''

archiveArtifacts 'legion_test/legion_test-pylint.log'
warnings canComputeNew: false, canResolveRelativePaths: false, categoriesPattern: '', defaultEncoding: '', excludePattern: '', healthy: '', includePattern: '', messagesPattern: '', parserConfigurations: [[ parserName: 'PyLint', pattern: 'legion_test/legion_test-pylint.log']], unHealthy: ''
archiveArtifacts 'legion-pylint.log'
step([
$class : 'WarningsPublisher',
parserConfigurations : [[
parserName: 'PYLint',
pattern : 'legion-pylint.log'
]],
unstableTotalAll : '99999',
usePreviousBuildAsReference: true
])
}
}
stage("Upload Legion package") {
Expand Down
2 changes: 1 addition & 1 deletion k8s/edge/Pipfile
Original file line number Diff line number Diff line change
Expand Up @@ -26,7 +26,7 @@ PyJWT = "==1.6.4"
nose = "==1.3.7"
pycodestyle = "==2.3.1"
pydocstyle = "==2.1.1"
pylint = "==1.7.2"
pylint = "==2.2.2"
tox = "==2.8.2"
locustio = "==0.8.1"
coverage = "==4.4.1"
Expand Down
43 changes: 7 additions & 36 deletions k8s/edge/Pipfile.lock

Some generated files are not rendered by default. Learn more about how customized files appear on GitHub.

2 changes: 1 addition & 1 deletion legion/requirements/Pipfile
Original file line number Diff line number Diff line change
Expand Up @@ -26,7 +26,7 @@ PyJWT = "==1.6.4"
nose = "==1.3.7"
pycodestyle = "==2.3.1"
pydocstyle = "==2.1.1"
pylint = "==1.7.2"
pylint = "==2.2.2"
tox = "==2.8.2"
locustio = "==0.8.1"
coverage = "==4.4.1"
Expand Down
8 changes: 4 additions & 4 deletions legion/requirements/Pipfile.lock

Some generated files are not rendered by default. Learn more about how customized files appear on GitHub.

2 changes: 1 addition & 1 deletion legion_airflow/requirements/Pipfile
Original file line number Diff line number Diff line change
Expand Up @@ -34,7 +34,7 @@ smart_open = "==1.5.7"
nose = "==1.3.7"
pycodestyle = "==2.3.1"
pydocstyle = "==2.1.1"
pylint = "==1.7.2"
pylint = "==2.2.2"
tox = "==2.8.2"
locustio = "==0.8.1"
coverage = "==4.4.1"
Expand Down
14 changes: 7 additions & 7 deletions legion_airflow/requirements/Pipfile.lock

Some generated files are not rendered by default. Learn more about how customized files appear on GitHub.

2 changes: 1 addition & 1 deletion legion_test/requirements/Pipfile
Original file line number Diff line number Diff line change
Expand Up @@ -32,7 +32,7 @@ pyOpenSSL = "==18.0.0"
nose = "==1.3.7"
pycodestyle = "==2.3.1"
pydocstyle = "==2.1.1"
pylint = "==1.7.2"
pylint = "==2.2.2"
tox = "==2.8.2"
locustio = "==0.8.1"
coverage = "==4.4.1"
Expand Down
14 changes: 7 additions & 7 deletions legion_test/requirements/Pipfile.lock

Some generated files are not rendered by default. Learn more about how customized files appear on GitHub.