diff options
| author | Christian Breunig <christian@breunig.cc> | 2025-09-04 21:35:37 +0200 |
|---|---|---|
| committer | Christian Breunig <christian@breunig.cc> | 2025-09-04 21:35:37 +0200 |
| commit | 5fee0201535fafceabd384ff3da3426e1fbd13fd (patch) | |
| tree | af6f4369dd1f96d56346f1b7d4a131fb2b9cca45 /Makefile | |
| parent | 4e0ed81965f0e3efd8054ccfc0403883864696e1 (diff) | |
| download | vyos-1x-5fee0201535fafceabd384ff3da3426e1fbd13fd.tar.gz vyos-1x-5fee0201535fafceabd384ff3da3426e1fbd13fd.zip | |
T7787: Makefile check for unused-imports should be mandatory
Currently our make target for "unused-imports" check is optional and a
"suggestion". As we now also use pylint to check for missing imports and code
issues, we will now make the unused import check mandatory.
This improves overall code quality.
Diffstat (limited to 'Makefile')
| -rw-r--r-- | Makefile | 6 |
1 files changed, 2 insertions, 4 deletions
@@ -119,6 +119,8 @@ check_migration_scripts_executable: pylint: interface_definitions @echo Running "pylint --errors-only ..." @PYTHONPATH=python/ pylint --errors-only $(shell git ls-files python/vyos/ifconfig/*.py python/vyos/utils/*.py src/conf_mode/*.py src/op_mode/*.py src/migration-scripts src/services/vyos*) + @echo Running "pylint to check for unused imports ..." + @PYTHONPATH=python/ pylint --disable=all --enable=W0611 $(shell git ls-files *.py src/migration-scripts src/services) .PHONY: j2lint j2lint: @@ -131,10 +133,6 @@ endif sonar: sonar-scanner -X -Dsonar.login=${SONAR_TOKEN} -.PHONY: unused-imports -unused-imports: - @pylint --disable=all --enable=W0611 $(shell git ls-files *.py src/migration-scripts src/services) - deb: dpkg-buildpackage -uc -us -tc -b |