Skip to content

Conversation

Copilot
Copy link
Contributor

@Copilot Copilot AI commented Jul 13, 2025

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:

  • Uses the required job name copilot-setup-steps for Copilot recognition
  • Sets up Python 3.12 and installs flit build system
  • Runs make install to install all project dependencies via flit
  • Follows GitHub Actions best practices with minimal permissions (contents: read)
  • Triggers on workflow dispatch and when the workflow file itself is modified
  • Contains no comments as requested in the issue

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.

Co-authored-by: lucasrcezimbra <7042915+lucasrcezimbra@users.noreply.github.com>
@Copilot Copilot AI changed the title [WIP] create copilot-setup-steps file Create copilot-setup-steps workflow for dependency installation Jul 13, 2025
@Copilot Copilot AI requested a review from lucasrcezimbra July 13, 2025 23:54
Copilot finished work on behalf of lucasrcezimbra July 13, 2025 23:54
@lucasrcezimbra lucasrcezimbra marked this pull request as ready for review July 14, 2025 00:50
@Copilot Copilot AI review requested due to automatic review settings July 14, 2025 00:50
Copy link
Contributor

@Copilot Copilot AI left a 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 runs make 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 with cancel-in-progress: true) to prevent duplicate workflow runs when multiple changes are pushed in quick succession.
name: "Copilot Setup Steps"

Copy link

codecov bot commented Jul 14, 2025

Codecov Report

All modified and coverable lines are covered by tests ✅

Project coverage is 94.73%. Comparing base (c701d5e) to head (d067b2e).
Report is 1 commits behind head on main.

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.
📢 Have feedback on the report? Share it here.

🚀 New features to boost your workflow:
  • ❄️ Test Analytics: Detect flaky tests, report on failures, and find test suite problems.

lucasrcezimbra and others added 2 commits July 13, 2025 23:55
Co-authored-by: Copilot <175728472+Copilot@users.noreply.github.com>
Co-authored-by: lucasrcezimbra <7042915+lucasrcezimbra@users.noreply.github.com>
Copilot finished work on behalf of lucasrcezimbra July 14, 2025 02:59
@Copilot Copilot AI requested a review from lucasrcezimbra July 14, 2025 02:59
@lucasrcezimbra
Copy link
Owner

pre-commit.ci autofix

Copy link
Owner

@lucasrcezimbra lucasrcezimbra left a 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 and pre-commit install-hooks to the make install
  • Remove the 'Cache pip packages' for now, because it's breaking the CI.

Copy link
Contributor Author

Copilot AI commented Jul 14, 2025

Actually, it seems that flit is required:

This error is happening in 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>
Copy link
Owner

@lucasrcezimbra lucasrcezimbra left a 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.

lucasrcezimbra and others added 2 commits July 14, 2025 08:25
Co-authored-by: lucasrcezimbra <7042915+lucasrcezimbra@users.noreply.github.com>
Copy link
Contributor Author

Copilot AI commented Jul 14, 2025

build/test using Python 3.8 is failing in the CI. Remove Python 3.8 from the CI since its end-of-life.

Removed Python 3.8 from the CI matrix as it's end-of-life and causing build failures. Commit d067b2e

@Copilot Copilot AI requested a review from lucasrcezimbra July 14, 2025 11:28
Copilot finished work on behalf of lucasrcezimbra July 14, 2025 11:28
@lucasrcezimbra lucasrcezimbra merged commit c64da92 into main Jul 14, 2025
9 checks passed
@lucasrcezimbra lucasrcezimbra deleted the copilot/fix-56 branch July 14, 2025 11:34
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

create copilot-setup-steps file

2 participants