-
Notifications
You must be signed in to change notification settings - Fork 2
Create copilot-setup-steps workflow for dependency installation #57
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
Conversation
Co-authored-by: lucasrcezimbra <7042915+lucasrcezimbra@users.noreply.github.com>
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.
Pull Request Overview
Adds a new GitHub Actions workflow to automate the Python development environment setup for the Copilot Coding Agent.
- Introduces
.github/workflows/copilot-setup-steps.yml
- Triggers on workflow dispatch, push, and pull request changes to this file
- Sets up Python 3.12, installs
flit
, and runsmake install
with minimal permissions
Comments suppressed due to low confidence (1)
.github/workflows/copilot-setup-steps.yml:1
- [nitpick] Consider adding a concurrency configuration (e.g.,
concurrency
withcancel-in-progress: true
) to prevent duplicate workflow runs when multiple changes are pushed in quick succession.
name: "Copilot Setup Steps"
Codecov ReportAll modified and coverable lines are covered by tests ✅
Additional details and impacted files@@ Coverage Diff @@
## main #57 +/- ##
=======================================
Coverage 94.73% 94.73%
=======================================
Files 11 11
Lines 266 266
=======================================
Hits 252 252
Misses 14 14 ☔ View full report in Codecov by Sentry. 🚀 New features to boost your workflow:
|
Co-authored-by: Copilot <175728472+Copilot@users.noreply.github.com>
for more information, see https://pre-commit.ci
Co-authored-by: lucasrcezimbra <7042915+lucasrcezimbra@users.noreply.github.com>
pre-commit.ci autofix |
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.
Actually, it seems that flit is required:
This error is happening in the CI:
Run make install
flit install --deps develop --symlink
make: flit: No such file or directory
make: *** [Makefile:4: install] Error 127
- Fix the make install error
- add
pre-commit install
andpre-commit install-hooks
to the make install - Remove the 'Cache pip packages' for now, because it's breaking the CI.
Fixed all three issues: re-added flit installation to resolve the make install error, added pre-commit setup to the install target, and removed the problematic cache step. The workflow should now properly install all dependencies including flit and pre-commit tools. Commit 6c9a4cd |
Co-authored-by: lucasrcezimbra <7042915+lucasrcezimbra@users.noreply.github.com>
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.
build/test using Python 3.8 is failing in the CI. Remove Python 3.8 from the CI since its end-of-life.
Co-authored-by: lucasrcezimbra <7042915+lucasrcezimbra@users.noreply.github.com>
Removed Python 3.8 from the CI matrix as it's end-of-life and causing build failures. Commit d067b2e |
This PR adds a GitHub Actions workflow file
.github/workflows/copilot-setup-steps.yml
to support GitHub Copilot Coding Agent's development environment setup.The workflow:
copilot-setup-steps
for Copilot recognitionmake install
to install all project dependencies via flitcontents: read
)The setup process has been tested and successfully installs all project dependencies including Django, django-ninja, pytest, and development tools.
Fixes #56.
💬 Share your feedback on Copilot coding agent for the chance to win a $200 gift card! Click here to start the survey.