Skip to content

Copier update (pre-commit workflow)#42

Merged
ejfine merged 7 commits intomainfrom
precommit-up
May 9, 2025
Merged

Copier update (pre-commit workflow)#42
ejfine merged 7 commits intomainfrom
precommit-up

Conversation

@ejfine
Copy link
Contributor

@ejfine ejfine commented May 9, 2025

Pull in upstream changes

@ejfine ejfine requested a review from Copilot May 9, 2025 18:42
@ejfine ejfine self-assigned this May 9, 2025
Copy link

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

This PR updates the Copier pre-commit workflow to pull in upstream changes and streamline CI job definitions. Key changes include adding a new pre-commit workflow in both template and repository directories, updating the CI job in ci.yaml/jinja to reuse the pre-commit workflow, and revising dependency update rules and Copier configuration.

Reviewed Changes

Copilot reviewed 6 out of 6 changed files in this pull request and generated no comments.

Show a summary per file
File Description
template/.github/workflows/pre-commit.yaml Adds a new pre-commit workflow with inputs for python and node versions.
template/.github/workflows/ci.yaml.jinja Replaces previous pre-commit steps with a reference to the new workflow.
.github/workflows/pre-commit.yaml Mirror update of the pre-commit workflow for the repository’s workflows.
.github/workflows/ci.yaml Refactors pre-commit job using the new workflow and now hardcodes the python version.
.github/dependabot.yml Introduces dependency update rules for sphinx* with minor and patch updates.
.copier-answers.yml Updates the Copier commit version.
Comments suppressed due to low confidence (1)

.github/workflows/ci.yaml:26

  • The pre-commit job now uses a fixed python-version (3.12.7) instead of leveraging a matrix as before. Confirm that multi-version testing is no longer required and update documentation if needed.
python-version: 3.12.7

@ejfine ejfine merged commit 97f8510 into main May 9, 2025
8 checks passed
@ejfine ejfine deleted the precommit-up branch May 9, 2025 18:44
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.

2 participants