-
Notifications
You must be signed in to change notification settings - Fork 0
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
[ENH] set up CI #1
Conversation
Reviewer's Guide by SourceryThis pull request sets up continuous integration (CI) for the project by adding pre-commit hooks, GitHub Actions workflows for building Docker and Apptainer images, and Dependabot configuration for automated dependency updates. The README.md file has also been updated to reflect the new project name and description. File-Level Changes
Tips
|
for more information, see https://pre-commit.ci
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.
Hey @Remi-Gau - I've reviewed your changes - here's some feedback:
Overall Comments:
- Consider preserving some of the original structural information in the README, adapting it to the new CAT12 container context if necessary.
Here's what I looked at during the review
- 🟡 General issues: 2 issues found
- 🟡 Security: 1 issue found
- 🟢 Testing: all looks good
- 🟢 Complexity: all looks good
- 🟡 Documentation: 1 issue found
Help me be more useful! Please click 👍 or 👎 on each comment to tell me if it was helpful.
Summary by Sourcery
This pull request sets up continuous integration (CI) for the project by adding pre-commit hooks, CI workflows for building Docker and Apptainer images, and Dependabot configuration for automatic dependency updates. Additionally, the README.md file has been updated to provide a clearer project description.