Skip to content

Runs are now failing Python Lint test in github actions CI #1823

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

Closed
bheiner14 opened this issue Aug 9, 2021 · 2 comments
Closed

Runs are now failing Python Lint test in github actions CI #1823

bheiner14 opened this issue Aug 9, 2021 · 2 comments

Comments

@bheiner14
Copy link
Contributor

After PR #1793 which dropped python 2.7 was merged into master yesterday, all runs are now failing the Python Lint test. According to the output, this test is failing due to lint errors in vtr_flow/scripts/run_vtr_task.py and no other files seem to have been affected.

@mithro
Copy link
Contributor

mithro commented Aug 9, 2021

vtr_flow/scripts/run_vtr_flow.py passed 
vtr_flow/scripts/run_vtr_task.py has lint errors 
************* Module run_vtr_task
vtr_flow/scripts/run_vtr_task.py:240:37: E1136: Value 'common_task_prefix' is unsubscriptable (unsubscriptable-object)
vtr_flow/scripts/download_symbiflow.py passed 
vtr_flow/scripts/arch_gen/arch_gen.py skipped (grandfathered) 
vtr_flow/scripts/python_libs/vtr/task.py passed 
vtr_flow/scripts/python_libs/vtr/error.py passed 
vtr_flow/scripts/python_libs/vtr/util.py passed 
vtr_flow/scripts/python_libs/vtr/__init__.py passed 
vtr_flow/scripts/python_libs/vtr/flow.py passed 
vtr_flow/scripts/python_libs/vtr/parse_vtr_task.py passed 
vtr_flow/scripts/python_libs/vtr/log_parse.py passed 
vtr_flow/scripts/python_libs/vtr/paths.py passed 
vtr_flow/scripts/python_libs/vtr/parse_vtr_flow.py passed 
vtr_flow/scripts/python_libs/vtr/abc/abc.py passed 
vtr_flow/scripts/python_libs/vtr/abc/__init__.py passed 
vtr_flow/scripts/python_libs/vtr/ace/ace.py passed 
vtr_flow/scripts/python_libs/vtr/ace/__init__.py passed 
vtr_flow/scripts/python_libs/vtr/vpr/__init__.py passed 
vtr_flow/scripts/python_libs/vtr/vpr/vpr.py passed 
vtr_flow/scripts/python_libs/vtr/odin/__init__.py passed 
vtr_flow/scripts/python_libs/vtr/odin/odin.py passed 
vtr_flow/scripts/spice/run_spice.py skipped (grandfathered) 
vtr_flow/scripts/benchtracker/populate_db.py skipped (grandfathered) 
vtr_flow/scripts/benchtracker/plotter-offline.py skipped (grandfathered) 
vtr_flow/scripts/benchtracker/util.py skipped (grandfathered) 
vtr_flow/scripts/benchtracker/__init__.py passed 
vtr_flow/scripts/benchtracker/server_db.py skipped (grandfathered) 
vtr_flow/scripts/benchtracker/interface_db.py skipped (grandfathered) 
vtr_flow/scripts/benchtracker/flask_cors/six.py skipped (grandfathered) 
vtr_flow/scripts/benchtracker/flask_cors/__init__.py passed 
vtr_flow/scripts/benchtracker/flask_cors/version.py skipped (grandfathered) 
vtr_flow/scripts/benchtracker/flask_cors/decorator.py skipped (grandfathered) 
vtr_flow/scripts/benchtracker/flask_cors/extension.py skipped (grandfathered) 
vtr_flow/scripts/benchtracker/flask_cors/core.py skipped (grandfathered) 
ERROR: 1 file(s) failed lint test. 
Error: Process completed with exit code 1.

@vaughnbetz
Copy link
Contributor

This was fixed in PR #1821

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

No branches or pull requests

3 participants