Merged
Conversation
* Update requirements and dockerfile * Use Dockerfile in Github Action * Ignore a part of flake8 issues * Ignore flake8 inspections * Separate docker file into two: for prod and for dev * Delete ` * Fix test_range_of_lines * Fix pylint and flake8 tests * Fix pylint tests * Check styles by flake8 * Fix C408 issue * Move production Dockerfile * Fix Dockefile for production * Set up Eslint in the DockerFile for production * Delete one case from flake8 tests * Add pathlib into whitelist * Delete Path from setup.py * Ignore pathlib * Don't count INFO issues * Fix flake8 tests * Small fix * Delete pathlib from the whitelist * Update docker (#54) * Use one docker file for prod and dev * Change base docker image * Fix requirements-test: delete duplicates, setup versions * Try to fix dockerfile * Fix run in subprocess function * Fix Dockerfile * Try to fix Dockerfile * Fix docker image (#58) * Change eslint installing * Update build.yaml * Introduce base docker image and auto-detect eslint path Co-authored-by: Andrey Balandin <andrvb@gmail.com> Closes #43, closes #45.
* Add whitelist for flake8-spellcheck * Add a test for spellcheck and ignore unnecessary rule * Fix flake8 config
* wps-light support (#17) Add wps-light support * New flake8 plugins support (#18) Added support for flake8-broken-line, flake8-string-format, flake8-commas (they are WPS dependencies). Added support for cohesion. * Radon support (#19) Added new inspector: Radon. Added maintainability index. * Requirements upgrade (#21) Updated versions of dependencies Added django dictionary support Fixed tests * xlsx-run-tool (#24) Added xlsx_tool_run.py – script to run the tool on multiple code samples stored in xlsx file * Update version to 1.2.0 * Rename output_format to format * Fix double quotes * Fix small issues * Fix small issues * Fix double quotes * Fix double quotes * Inspectors fix (#39) * Add origin class for maintainability index * Add WPS518 to ignore due to collision with C0200 by Pylint * New category (#38) * Added a new category INFO, in order not to take into account some issues in the evaluation. * Fixed trailing commas * Delete create_directory function * Fix GitHub actions (#44) * Use the same environment as in the production dockerfile * Fix Dockerfiles * Install some dependencies for TeamCity * Update build * Add checks from teamcity * Update build * Fix indentation * Fix PR comments * Delete init files from resources * Add init in multi file project * Fix flake8 tests * Main upd bugs fix (#74) * Delete evaluation part * Fix bugs with detekt, pmd and flake8 * Issues fix (#80) * Fixed issue #70 * Fixed issue #72: now only spellcheck checks brackets * Fixed issue #73 * Fixed issue #77 * Fixed issue #78 * Fixed issue #71 * Fixed issue #79 * Fixed issue #81 * Added case 36: unpacking * Added case 37: wildcard import * Ignoring WPS347 and F405 * Added cases 36 (unpacking) and 37 (wildcard_import). Also removed W0622 (redefining_builtin) * Removed W0622 (redefining_builtin) * Issue #87 fix (#90) * Fixed issue #87 * Added test * Fixed issues #91 and #92 (#97) * Delete xlsx * Remove openpyxl * Issues fix (#104) * Fixed #100 * Fixed #102 * Added some more new words * Added toplevel * Update flake8 whitelist * Recovered accidentally deleted words Co-authored-by: Vlasov Ilya <55441714+GirZ0n@users.noreply.github.com> Co-authored-by: Daria Diatlova <dari.diatlova@gmail.com> Co-authored-by: Nikolay Vyahhi <Nikolay.Vyahhi@stepik.org> Co-authored-by: Ilya Vlasov <ilyavlasov2011@gmail.com>
# Conflicts: # .github/workflows/build.yml # VERSION.md # requirements-test.txt # requirements.txt # src/python/common/tool_arguments.py # src/python/review/application_config.py # src/python/review/common/file_system.py # src/python/review/inspectors/detekt/detekt.py # src/python/review/inspectors/eslint/eslint.py # src/python/review/inspectors/flake8/.flake8 # src/python/review/inspectors/flake8/flake8.py # src/python/review/inspectors/intellij/intellij.py # src/python/review/inspectors/issue.py # src/python/review/inspectors/parsers/checkstyle_parser.py # src/python/review/inspectors/pmd/files/bin/basic.xml # src/python/review/inspectors/pmd/issue_types.py # src/python/review/inspectors/pmd/pmd.py # src/python/review/inspectors/pyast/python_ast.py # src/python/review/inspectors/pylint/pylint.py # src/python/review/inspectors/radon/radon.py # src/python/review/inspectors/spotbugs/spotbugs.py # src/python/review/inspectors/springlint/springlint.py # src/python/review/reviewers/common.py # src/python/review/reviewers/perform_review.py # src/python/review/reviewers/utils/code_statistics.py # src/python/review/reviewers/utils/print_review.py # src/python/review/run_tool.py # test/python/functional_tests/conftest.py # test/python/functional_tests/test_multi_file_project.py # test/python/functional_tests/test_range_of_lines.py # test/python/functional_tests/test_single_file_json_format.py # test/python/inspectors/test_checkstyle_inspector.py # test/python/inspectors/test_flake8_inspector.py # test/python/inspectors/test_local_review.py # test/python/inspectors/test_pmd_inspector.py # test/resources/inspectors/python/case37_wildcard_import.py # whitelist.txt
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
2 participants
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
No description provided.