Develop #191
Merged
Develop #191
Conversation
This file contains 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
* Add LanguageVersion.JAVA_17 * Add B007's custom tip * Add W1404 and R1721' custom tips * Add new words * Add some inspections in ignore * Sort whitelists (Github Actions) * Add WPS442 * Remove W0621 * Add java17 * Update PMD to 6.47.0 * Add java17 * Update PMD to 6.47.0 * Update PMD to 6.47.0 * Remove unnecessary whitespaces * Add test for Java 17 * Bump version 1.2.7 -> 1.2.8 * Add trailing new line * Try to update docker image * Bump version 1.2.8 -> 1.3.0 Co-authored-by: GirZ0n <[email protected]> Co-authored-by: Anastasiia.Birillo <[email protected]>
* Add LanguageVersion.JAVA_17 * Add B007's custom tip * Add W1404 and R1721' custom tips * Add new words * Add some inspections in ignore * Sort whitelists (Github Actions) * Add WPS442 * Remove W0621 * Add flake8 data folder * Add is_fstring function * Add `get_measure_name_by_measurable_issue_type` function * Add metric codes * Refactor some strings into format strings * Add new words * Fix flake8 issues * Add IssueConfig, IssueDescriptionParser and IssueConfigsHandler * Refactor flake8 using new IssueConfigsHandler class * Add issue configs tests * Sort whitelists (Github Actions) * Typo fix * Typo fix * Typo fix * Remove unnecessary tests and add some more ones * Add one more test * Rename test data * Remove unnecessary tests and add some more * Move augmented assign pattern tip to `tips.py` * Remove unnecessary dicts * Require the `new_description` field * Fix tests * Accept only a dictionary in IssueDescriptionParser * Fix tests * Add new tests * Add new tests * Fix problems with escaped characters * Move some common files into common folder * Add new pylint's tests * Refactor pylint * Fix imports * Fix imports * Move common code into a new function * Refactor pmd * Refactor checkstyle and add new tests * Update tips * Refactor xml parser * Fix tests * Fix description * Remove unused impoty * Change default argument for get_magic_number_tip * Refactor detekt * Fix import style * Add new tests for pylint * Refactor eslint * Add new tests for eslint * Bump version 1.3.0 -> 1.3.1 * Update todo * Fix comment * Fix import * Change the order of checks in __post_init__ and fix typos * Add new tests * Remove unnecessary typing * Fix import style * Try to update docker image Co-authored-by: GirZ0n <[email protected]> Co-authored-by: Anastasiia.Birillo <[email protected]>
* Add Golang config * Add Go language * Add new words * Add Go language * Add golangci-lint inspector * Add golang and golangci-lint * Sort whitelists (Github Actions) * Add new words * Add metric linter names * Sort whitelists (Github Actions) * Add lightbulb * Sort whitelists (Github Actions) * Sort whitelists (Github Actions) * Fix #151: disable AbstractClassWithoutAbstractMethod. * Undo 6c8d506 * Add get_extension_from_file * Fix the bug that caused the tool to fail * Add test * Fix code style * Script for setting up the environment (#155) * Add a script for setting up the environment * Fix typo * Update README * Update README * Fix PR issues * Fix extensions * Update VERSION.md * Add WIP config * Fix golangci-lint's download command * Add golangci-lint download * Add working directory argument * Add ability to analyze folders * Fix extension * Fix typo * Fix extension * Update golangci-lint to 1.47.1 * Fix generics * Fix incorrect opening of Go projects without go.mod files * Add concurrency flag * Add issue types * Fix typo * Add noqa flag * Fix #156 (#157) * Add get_extension_from_file * Fix the bug that caused the tool to fail * Add test * Fix code style * Fix extensions * Update VERSION.md * Add logging * Move go installation into Dockerfile.base * Add --allow-parallel-runners flag * Fix curl flags * Fix typo * Fix configs * Move is_result_file_correct to common utils * Rename GolangLint -> GolangLintInspector * Small fixes: 1. Rename GolangLint -> GolangLintInspector; 2. Add a check for an invalid output file 3. Fix description regexp 4. Fix description style 5. Fix comment * Add some golang tests * Add a trailing new line * Fix typo * Remove gocognit * Add go data folder * Add tests * Fix typo * Add `n_line_len` and `n_info` * Add some more tests * Rename a variable * Fix flake8 issue * Add some more tests * Bump version to 1.4.0 * Added a new test case * Remove an unnecessary field * Remove an unnecessary field and replace `deadcode` with `unused` * Bump `golangci-lint` version: 1.47.1 -> 1.49.0 * Remove deprecated linters * Fix tests * Add new linter * Fix config * Fix import and measure's descriptions * Fix import * Move `is_result_file_correct` function to `utils.py` * Move inspector to a new architecture * Add new tests * Add issue configs * Fix config * Fix config * Fix typo * Add installation of third party golang libraries * Add _is_metalinter_issue * Remove blank line * Try to update docker image * Change container Co-authored-by: GirZ0n <[email protected]> Co-authored-by: Anastasiia.Birillo <[email protected]>
# Conflicts: # .github/workflows/build.yml # VERSION.md # hyperstyle/src/python/review/inspectors/common/utils.py # hyperstyle/src/python/review/inspectors/common/xml_parser.py # test/python/inspectors/__init__.py
* Create CONTRIBUTING.md * Update CONTRIBUTING.md * Update CONTRIBUTING.md * Update CONTRIBUTING.md * Update CONTRIBUTING.md * Fix link * Update CONTRIBUTING.md
* Try to add in memory api * Fix requirements conflicts * Fix flake8 * Add inspect_in_memory fo inspectors * Try to support in memory inspectors * Fix styles * Fix styles and add more helpers * Try to adapt flake8 and run_in_subprocess function * Try to fix teamcity tests * Fix flake * Try to fix teamcity * Try to fix teamcity * Fix tests * added inspect_in_memory for pyling inspector (#168) * added inspect_in_memory for pyling inspector * fix formatting * Fix tests Co-authored-by: Anastasiia.Birillo <[email protected]> * Update version Co-authored-by: Sergey Titov <[email protected]>
# Conflicts: # VERSION.md # hyperstyle/src/python/review/common/subprocess_runner.py # hyperstyle/src/python/review/inspectors/golang_lint/golang_lint.py
* Add simple ij inspector * Move code server params to env variables and update Dockerfile * Fix flake8 * Fix flake8 * Fix flake8 * Fix flake8 * Sort whitelists (Github Actions) * Try to fix tests * Fix code style * Wrap server connection with try/catch * Remove test main * Fix tests * Edit README.md * Fix review comments Co-authored-by: tiginamaria <[email protected]> Co-authored-by: Anastasiia.Birillo <[email protected]>
* Disable whitespace inspection for Go and fix a small error in README * Fix tests * Update version
# Conflicts: # VERSION.md
* Fix ij-inspector * Fix issues * Fix styles
* fix: fix WPS link * fix: fix golangci-lint link * fix: fix golangci-lint link * fix: fix PMD link * fix: fix Detekt link * fix: fix checkstyle link
* fix: update .gitignore to exclude unnecessary files * feat: add IJ Kotlin inspector feat: add the `ij-config` argument feat: add BaseIJInspector refactor: move several inspector-related files to the common folder * fix: update flake8 run command * docs: refined the `--ij-config` argument description * fix: update test configuration to disable IJ_PYTHON and IJ_KOTLIN * fix: add the `ij_config` argument to test command builder * fix: fix tests * fix: add IJ_KOTLIN to an inspector types list * style: fix flake8's issues * refactor: rename variable for clarity in common.py * style: fix flake8's issue * docs: rearrange wording for consistent format in README * fix: fix PR issues * style: fix flake8's issues * fix: bump the version number
* fix: update .gitignore to exclude unnecessary files * feat: add IJ Kotlin inspector feat: add the `ij-config` argument feat: add BaseIJInspector refactor: move several inspector-related files to the common folder * fix: update flake8 run command * docs: refined the `--ij-config` argument description * fix: update test configuration to disable IJ_PYTHON and IJ_KOTLIN * fix: add the `ij_config` argument to test command builder * fix: fix tests * fix: add IJ_KOTLIN to an inspector types list * style: fix flake8's issues * refactor: rename variable for clarity in common.py * style: fix flake8's issue * docs: rearrange wording for consistent format in README * refactor: refactor build system * fix: try to fix workflows * fix: try to fix workflows * style: fix flake8's issue * fix: remove MANIFEST.in * fix: update project license type and remove unneeded sections * feat: update setup.py for generating proto files * fix: add protobuf compilation step to build workflow * bump: bump the tool version * fix: update a proto path in build workflow * fix: add proto generation to Dockerfile * fix: fix incorrect ENV syntax in Dockerfile * fix: update PROTO_PATH in Dockerfile * refactor: refactor dependencies structure * fixL fix a comment * refactor: refactor Dockerfile, optimizing sequence of commands * feat: add new directories to .dockerignore * fix: return dependencies installation * fix: specify versions for setuptools and setuptools-scm * style: remove unnecessary cd * fix: remove redundant dependencies * fix: removed unnecessary step * fix: fix the GitHub workflow * fix: update project author in pyproject.toml * docs: update README.md installation instructions * fix: update the build system requires and exclude unnecessary files
…as always `IJ_PYTHON` (#188)
# Conflicts: # VERSION.md # test/resources/inspectors/python/case39_no_issues.py
* Switch to Poetry for dependency management Replaced `requirements` files and `setup.py` with `pyproject.toml` for Poetry. Updated Dockerfile to use Poetry for project build and dependency installation, and added a new GitHub Actions workflow for sorting whitelists. * Switch workflows to self-hosted runners Updated the `.github/workflows/sort_whitespaces.yml`, `.github/workflows/publish.yml`, and `.github/workflows/build.yml` files to use self-hosted runners with a small configuration. This change aims to leverage available self-hosted resources for running GitHub Actions. * Upgrade grpcio-tools and protobuf versions Upgraded grpcio-tools from 1.51.1 to 1.66.1 and protobuf from 4.25.4 to 5.28.1 in pyproject.toml and poetry.lock to the latest versions. Updated Dockerfile to adjust paths and improve build context. These changes improve compatibility and maintain security standards. * Update Python version to 3.10.14 Switching from Python 3.8.11 to 3.10.14 for improved features and performance enhancements. Ensure compatibility with the newer version in your development environment. * Update wps-light to version 0.15.3 This commit upgrades the wps-light dependency from version 0.15.2 to 0.15.3 in the pyproject.toml file. This ensures the project benefits from the latest fixes and improvements provided in the newer version. * Use sys.executable and refactor Dockerfile for venv Switched to using sys.executable in multiple files for better flexibility with the Python interpreter. Updated the Dockerfile to create and use a virtual environment, and adjusted related workflow scripts to use the virtual environment's executables. Added a new GitHub Actions workflow for building the Docker image after a release. --------- Co-authored-by: meanmail <[email protected]>
The VERSION.md file has been deleted to streamline project maintenance. Version tracking will now rely on internal systems rather than a separate file.
added 4 commits
September 19, 2024 16:01
Replaced the custom `remove_prefix` function with Python's built-in `removeprefix` method for better readability and efficiency. This change modernizes the code and leverages Python's built-in capabilities.
Removed the default 'ij_python' inspector from the disabled set in the `run_tool.py` file. This change prepares the code for production by ensuring no inspectors are disabled by default.
Reverting the version number from 1.5.1 back to 1.5.0. This change rectifies an unintended version increment in the previous update.
Remove the `remove_prefix` import from `pmd.py` which is not being used in the code. This helps to clean up the code and avoid unnecessary imports.
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
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.
Issues:
Issue#
Description: