Skip to content

Create pre-commit.yml#369

Open
tschm wants to merge 6 commits intoplotly:masterfrom
tschm:patch-6
Open

Create pre-commit.yml#369
tschm wants to merge 6 commits intoplotly:masterfrom
tschm:patch-6

Conversation

@tschm
Copy link
Copy Markdown
Contributor

@tschm tschm commented Jul 26, 2025

This should not pass at this stage as the problems with the README.md file are addressed in a different branch

This should not pass at this stage as the problems with the README.md file are addressed in a different branch
@gvwilson
Copy link
Copy Markdown
Contributor

gvwilson commented Aug 1, 2025

See #368.

@gvwilson gvwilson requested a review from ayjayt August 1, 2025 14:41
@gvwilson gvwilson added feature something new community community contribution P2 needed for current cycle labels Aug 1, 2025
Copilot AI review requested due to automatic review settings May 2, 2026 18:25
Copy link
Copy Markdown

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 GitHub Actions workflow to run the repository’s pre-commit checks in CI (using the config in src/py/.pre-commit-config.yaml).

Changes:

  • Introduces a new pre-commit GitHub Actions workflow.
  • Checks out the repo, sets up Node, and runs pre-commit/action across all files using the Python subproject’s pre-commit config.

💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.

Comment on lines +3 to +5
# Trigger: This workflow runs on every push to the repository
on:
push:
Comment on lines +3 to +5
# Trigger: This workflow runs on every push to the repository
on:
push:
Comment on lines +20 to +24
- name: Install Node 22
uses: actions/setup-node@v4
with:
node-version: '22'

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

community community contribution feature something new P2 needed for current cycle

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants