Skip to content

gh-7: Add CI/CD workflow for pre-commit and tests - #8

Merged
gcanasherrera merged 6 commits into
mainfrom
copilot/add-ci-cd-setup
Feb 8, 2026
Merged

gcanasherrera merged 6 commits into
mainfrom
copilot/add-ci-cd-setup

Conversation

Copilot AI commented Feb 6, 2026 •

Copy link
Copy Markdown
Contributor

CI/CD Setup for Pre-commit and Tests

  • Create GitHub Actions workflow for CI/CD
    • Add workflow to run pre-commit hooks
    • Add workflow to run tests (pytest)
    • Configure workflow to run on push and pull requests
  • Create .gitignore to exclude build artifacts
  • Validate workflow syntax
  • Add explicit permissions to limit GITHUB_TOKEN (security best practice)
  • Exclude notebooks from pre-commit hooks
  • Fix formatting in release.yml (remove trailing whitespace)
  • Handle pytest exit code 5 (no tests found) to prevent CI failure
  • Add comment explaining exit code handling
  • All security checks passed (CodeQL)
  • Code review completed with no issues
Original prompt

This section details on the original issue you should resolve

<issue_title>Add CI/CD</issue_title>
<issue_description></issue_description>

<agent_instructions>I would like to run pre-commit and tests in CI/CD</agent_instructions>

Comments on the Issue (you are @copilot in this section)


💡 You can make Copilot smarter by setting up custom instructions, customizing its development environment and configuring Model Context Protocol (MCP) servers. Learn more Copilot coding agent tips in the docs.

Copilot AI and others added 2 commits February 6, 2026 22:34
Co-authored-by: gcanasherrera <13239454+gcanasherrera@users.noreply.github.com>
Co-authored-by: gcanasherrera <13239454+gcanasherrera@users.noreply.github.com>
Copilot AI changed the title [WIP] Add CI/CD for pre-commit and test runs Add CI/CD workflow for pre-commit and tests Feb 6, 2026
Copilot AI requested a review from gcanasherrera February 6, 2026 22:37
@gcanasherrera gcanasherrera changed the title Add CI/CD workflow for pre-commit and tests gh-7: Add CI/CD workflow for pre-commit and tests Feb 8, 2026
@gcanasherrera

Copy link
Copy Markdown
Member

@copilot, please avoid running pre-commit in notebooks

Co-authored-by: gcanasherrera <13239454+gcanasherrera@users.noreply.github.com>

Copilot AI commented Feb 8, 2026

Copy link
Copy Markdown
Contributor Author

@copilot, please avoid running pre-commit in notebooks

Done! Added exclude: ^notebooks/ to all relevant pre-commit hooks. Notebooks will now be skipped during pre-commit checks in CI and locally. (commit 75c9305)

@gcanasherrera

Copy link
Copy Markdown
Member

@copilot please fix the error in pre-commit

Copilot AI and others added 2 commits February 8, 2026 21:18
Co-authored-by: gcanasherrera <13239454+gcanasherrera@users.noreply.github.com>
Co-authored-by: gcanasherrera <13239454+gcanasherrera@users.noreply.github.com>

Copilot AI commented Feb 8, 2026

Copy link
Copy Markdown
Contributor Author

@copilot please fix the error in pre-commit

Fixed! The pre-commit errors were caused by:

  1. Trailing whitespace in release.yml - removed
  2. Pytest failing with exit code 5 when no tests exist - now handled gracefully

CI should pass now. (commits 58138fe, 892286b)

@gcanasherrera
gcanasherrera marked this pull request as ready for review February 8, 2026 21:20
@gcanasherrera
gcanasherrera merged commit 1bac3b7 into main Feb 8, 2026
6 checks passed
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.

Add CI/CD

2 participants