-
Notifications
You must be signed in to change notification settings - Fork 1
feat: add pyansys-quality-check hook
#486
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
Draft
Revathyvenugopal162
wants to merge
55
commits into
main
Choose a base branch
from
feat/pyansys-quality-report
base: main
Could not load branches
Branch not found: {{ refName }}
Loading
Could not load tags
Nothing to show
Loading
Are you sure you want to change the base?
Some commits from the old base branch may be removed from the timeline,
and old review comments may become outdated.
Draft
Changes from all commits
Commits
Show all changes
55 commits
Select commit
Hold shift + click to select a range
6806db9
feat: pyansys-quality-report hook
Revathyvenugopal162 d331259
feat: migrate the checks
Revathyvenugopal162 9edff47
feat: migrate the checks
Revathyvenugopal162 118166b
feat: show all checks args
Revathyvenugopal162 f16d9a5
chore: adding changelog file 486.added.md [dependabot-skip]
pyansys-ci-bot 48e39dc
feat: keep checks in seperate modules
Revathyvenugopal162 4d58454
Merge branch 'feat/pyansys-quality-report' of https://github.com/ansy…
Revathyvenugopal162 e3f1f1c
chore: auto fixes from pre-commit hooks
pre-commit-ci[bot] 36d2ef3
fix: add subpackage
Revathyvenugopal162 9ef11ee
fix: add subpackage
Revathyvenugopal162 aee0f08
Merge branch 'feat/pyansys-quality-report' of https://github.com/ansy…
Revathyvenugopal162 36e0c3b
chore: auto fixes from pre-commit hooks
pre-commit-ci[bot] d370d11
Merge branch 'main' into feat/pyansys-quality-report
Revathyvenugopal162 662f1c5
fix: merge the old tech review for bootsrap mode
Revathyvenugopal162 613748b
Merge branch 'feat/pyansys-quality-report' of https://github.com/ansy…
Revathyvenugopal162 2ab095d
chore: auto fixes from pre-commit hooks
pre-commit-ci[bot] 29fd50c
fix: merge the old tech review for bootsrap mode
Revathyvenugopal162 0735f48
Merge branch 'feat/pyansys-quality-report' of https://github.com/ansy…
Revathyvenugopal162 4e941ab
fix: pre-commit
Revathyvenugopal162 8e1545b
fix: merge old tech review
Revathyvenugopal162 6a4e202
fix: merge old tech review
Revathyvenugopal162 a5fe847
chore: auto fixes from pre-commit hooks
pre-commit-ci[bot] a15c09c
fix: codestyle
Revathyvenugopal162 1769bf9
Merge branch 'feat/pyansys-quality-report' of https://github.com/ansy…
Revathyvenugopal162 b114f14
fix: codes
Revathyvenugopal162 fb800d8
chore: auto fixes from pre-commit hooks
pre-commit-ci[bot] a8344e7
fix: codestylee
Revathyvenugopal162 7d4157d
Merge branch 'feat/pyansys-quality-report' of https://github.com/ansy…
Revathyvenugopal162 9890fc3
fix: codestylee
Revathyvenugopal162 507c0ed
fix: updatedoc-build errpr
Revathyvenugopal162 da5b0b7
fix: update license
Revathyvenugopal162 cfb7b11
fix: update license
Revathyvenugopal162 1a34264
fix: update packages
Revathyvenugopal162 0cfb897
chore: auto fixes from pre-commit hooks
pre-commit-ci[bot] 2feea01
fix: update packages
Revathyvenugopal162 962478e
Merge branch 'feat/pyansys-quality-report' of https://github.com/ansy…
Revathyvenugopal162 85177d1
fix: build backend module
Revathyvenugopal162 e1128da
fix: ci_cd files modules
Revathyvenugopal162 4ed5b30
fix: cicd module and sematic versioning
Revathyvenugopal162 ca4a453
fix: common module
Revathyvenugopal162 a1804e9
fix: dependabot module
Revathyvenugopal162 f844b64
fix: documentation module
Revathyvenugopal162 cf47ad1
fix: pre-commit and labeler
Revathyvenugopal162 7dacc8a
fix: project metadata module
Revathyvenugopal162 5791a2b
fix: readme module
Revathyvenugopal162 05a8c21
fix: security module and also all the docstrings
Revathyvenugopal162 61447b8
fix: vale module
Revathyvenugopal162 78cb6f8
feat: add additional tests
Revathyvenugopal162 fe736c9
feat: add additional metadata
Revathyvenugopal162 f911449
feat: add additional metadata tests
Revathyvenugopal162 606ba93
feat: add configuration
Revathyvenugopal162 8712228
fix: suggestions by @jorgepiloto
Revathyvenugopal162 e48c33b
fix: suggestions by @jorgepiloto
Revathyvenugopal162 ed1e8c1
fix: suggestions by @jorgepiloto
Revathyvenugopal162 9c5d29b
fix: revert tech-review precommit
Revathyvenugopal162 File filter
Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
There are no files selected for viewing
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
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
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
| Original file line number | Diff line number | Diff line change |
|---|---|---|
| @@ -0,0 +1 @@ | ||
| Add \`pyansys-quality-check\` hook |
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
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
Oops, something went wrong.
Oops, something went wrong.
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.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Blocking: this replaces the
tech-reviewhook id instead of addingpyansys-quality-reportalongside it.setup.pydrops thetech-reviewconsole script too. Any config pinned toid: tech-reviewbreaks, including this repo's own.pre-commit-config.yaml:62. Keep the old id/script (deprecated is fine) or treat this as an intentional breaking change and update the config.There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Now It is using V8 of pre-commit hook. I have to reformat to local, to replace this new hook. i will do that.