chore: optimize project metadata and CI/CD#1
Conversation
![chatgpt-codex-connector[bot]](https://avatars.githubusercontent.com/in/1144995?s=60&v=4){kind=small}
There was a problem hiding this comment.
๐ก Codex Review
Here are some automated review suggestions for this pull request.
Reviewed commit: fe70eb1423
โน๏ธ About Codex in GitHub
Your team has set up Codex to review pull requests in this repo. Reviews are triggered when you
- Open a pull request for review
- Mark a draft as ready
- Comment "@codex review".
If Codex has suggestions, it will comment; otherwise it will react with ๐.
Codex can also answer questions or update the PR. Try commenting "@codex address that feedback".
| - name: Lint with ruff | ||
| run: | | ||
| pip install ruff | ||
| ruff check skills/ --ignore E501,F401 |
There was a problem hiding this comment.
In this workflow I ran the exact command added here (ruff check skills/ --ignore E501,F401); it exits 1 with 63 existing violations, including F821 Undefined name argparse in skills/ppt-master/scripts/svg_finalize/crop_images.py. Since Ruff's help says it runs on the supplied files/directories, every push/PR matrix job will stop at this new step until the repo is fixed, the rule set is tuned, or the check is scoped to code that already passes.
Useful? React with ๐ย / ๐.
| - name: Format check with black | ||
| run: | | ||
| pip install black | ||
| black --check skills/ --line-length 100 |
There was a problem hiding this comment.
After the Ruff step is addressed, this newly added Black gate still fails on the current repository: black --check skills/ --line-length 100 exits 1 and reports 53 files that would be reformatted. Black's --check help documents that return code 1 means files would change, so this makes all CI jobs fail until the existing files are formatted or this check is narrowed/removed.
Useful? React with ๐ย / ๐.
| - name: Run basic validation | ||
| run: | | ||
| python -c "import sys; print(f'Python {sys.version}')" | ||
| python -c "from skills.ppt_master.scripts import svg_to_pptx; print('Import successful')" |
There was a problem hiding this comment.
This validation command cannot succeed from a fresh checkout because the repository contains skills/ppt-master, not an importable skills/ppt_master package; running the exact command raises ModuleNotFoundError: No module named 'skills.ppt_master'. Once the earlier quality gates are fixed, every matrix job will still fail here unless the check runs the script by path or a real Python package is added.
Useful? React with ๐ย / ๐.
1 participant
Summary
This PR adds essential project infrastructure and automation to improve maintainability and contributor experience.
Changes
โ GitHub Actions CI/CD
โ Version Tracking
โ Issue Templates
โ Pull Request Template
โ License Update
Impact
Testing
All new files follow GitHub's standard formats:
Roadmap Progress
This PR completes the following roadmap items:
Notes