Repository navigation
GitHub actions to check formatting - #265
Conversation
![github-advanced-security[bot]](https://avatars.githubusercontent.com/in/57789?s=60&v=4){kind=small}
…n permissions Co-authored-by: Copilot Autofix powered by AI <62310815+github-advanced-security[bot]@users.noreply.github.com>
…om/yaptide/converter into github-actions-to-check-formatting
There was a problem hiding this comment.
Pull request overview
Adds a GitHub Actions workflow to run the repository’s pre-commit hooks in CI on pushes to master and on pull requests, improving consistency of formatting/lint checks before merge.
Changes:
- Added
.github/workflows/pre-commit.ymlto set up Python/Poetry and runpre-commit run --all-filesin CI. - Normalized
.git-blame-ignore-revsfile formatting (adds trailing newline).
Reviewed changes
Copilot reviewed 1 out of 2 changed files in this pull request and generated 4 comments.
| File | Description |
|---|---|
| .github/workflows/pre-commit.yml | New workflow to install dependencies and run all pre-commit hooks in CI. |
| .git-blame-ignore-revs | Ensures the file has a proper trailing newline / consistent formatting. |
💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.
Co-authored-by: Copilot <175728472+Copilot@users.noreply.github.com>
Co-authored-by: Copilot <175728472+Copilot@users.noreply.github.com>
There was a problem hiding this comment.
Pull request overview
Copilot reviewed 2 out of 4 changed files in this pull request and generated 1 comment.
💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.
|
I'd suggest getting rid of duplicated dependencies in pyproject and simply |
46e75b4 to
8749105
Compare
…n permissions Co-authored-by: Copilot Autofix powered by AI <62310815+github-advanced-security[bot]@users.noreply.github.com>
There was a problem hiding this comment.
Pull request overview
Copilot reviewed 1 out of 2 changed files in this pull request and generated 1 comment.
💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.
Done in commit |
74ba143 to
7adb516
Compare
|
@grzanka it should be alright now |
There was a problem hiding this comment.
Pull request overview
Copilot reviewed 1 out of 2 changed files in this pull request and generated 2 comments.
💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.
Co-authored-by: Leszek Grzanka <grzanka@agh.edu.pl>
Co-authored-by: Leszek Grzanka <grzanka@agh.edu.pl>
Co-authored-by: Copilot <175728472+Copilot@users.noreply.github.com>
This pull request adds a new GitHub Actions workflow to automate pre-commit checks on every push to the
masterbranch and on pull requests. The workflow ensures that code quality checks are consistently run in CI before changes are merged.Continuous integration improvements:
.github/workflows/pre-commit.ymlworkflow that installspre-commitviapip, sets up Python 3.12, and runs all pre-commit hooks on every push tomasterand on pull requests.permissions: contents: readto the workflow to follow least-privilege security practices.