-
Notifications
You must be signed in to change notification settings - Fork 7
Styx-ci-cd #6
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
base: KAVR_12182025
Are you sure you want to change the base?
Styx-ci-cd #6
Conversation
Using Python 3.11.11 environment at: .kev
|
|
@sebastianorbell-qm I got stuck here / I could not run pre-commit from powershell not sure if I have to do it somewhere else. I had staged changes in git I made tiny comment on it
|
- Move .pre-commit-config.yaml, .pylintrc, and CONTRIBUTING.md to superconducting/ - Update pyproject.toml to reference .pylintrc in same directory - Update CONTRIBUTING.md with correct repository paths This consolidates all configuration files at the same level as pyproject.toml for better organization and easier maintenance.
Pylint automatically discovers .pylintrc in the same directory, so the [tool.pylint.main] section is unnecessary.
|
@KevinAVR I have updated the instructions in CONTRIBUTING.md and moved the pre-commit yaml into the same directory as pyproject.toml |
Add standard Python project exclusions including: - Virtual environments (.venv/, venv/) - Python cache files (*.pyc, __pycache__/) - Test artifacts (.coverage, htmlcov/) - IDE files (.vscode/, .idea/, .DS_Store) - Build artifacts (dist/, build/, *.egg-info/) - Tool caches (.mypy_cache/, .ruff_cache/, .pre-commit-cache/) Note: uv.lock is intentionally tracked for reproducible builds.
OziEgri
left a comment
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.
It looks good, was it tested E2E yet?

chore: Add CI/CD setup with Black, Pylint, and pre-commit hooks to project styx
PR Description:
Summary
Changes
Configuration Files Added
.pre-commit-config.yaml: Pre-commit hooks configuration with:.pylintrcconfiguration.pylintrc: Pylint configuration enabling errors, refactor suggestions, and conventions while disabling warningsCONTRIBUTING.md: Comprehensive contribution guide covering:pyproject.toml Updates
[tool.pylint.main]to reference.pylintrc[tool.pytest.ini_options]with coverage settings[tool.mypy]configuration[tool.commitizen]for version managementDeveloper Setup
After merging, developers should run: