Skip to content

ci(nix): run flake-check only when nix files change#190

Merged
ryoppippi merged 2 commits into
mainfrom
ci/flake-check-paths-filter
Dec 9, 2025
Merged

ci(nix): run flake-check only when nix files change#190
ryoppippi merged 2 commits into
mainfrom
ci/flake-check-paths-filter

Conversation

@ryoppippi

@ryoppippi ryoppippi commented Dec 9, 2025

Copy link
Copy Markdown
Contributor

Summary

  • Add paths filter to pull_request trigger so flake-check only runs when nix-related files are changed
  • Fix workflow file extension reference from .yml to .yaml for consistency

Changes

The flake-check workflow now only triggers on PRs that modify:

  • flake.nix
  • flake.lock
  • .github/workflows/nix-flake.yaml
  • .github/actions/setup-nix/**

This reduces unnecessary CI runs on PRs that don't modify nix files.

Test plan

  • Verify workflow syntax is valid
  • Create a PR without nix file changes and confirm flake-check doesn't run

Summary by cubic

Update flake-check CI to run only when Nix-related files change, reducing unnecessary runs. Also fix the workflow path to use .yaml.

  • New Features

    • Add a paths filter to the pull_request trigger for flake.nix, flake.lock, .github/workflows/nix-flake.yaml, and .github/actions/setup-nix/**.
  • Bug Fixes

    • Correct the push paths reference from .yml to .yaml.

Written for commit c0a5227. Summary will update automatically on new commits.

- Add paths filter to pull_request trigger so flake-check only runs
  when nix-related files are changed
- Fix workflow file extension from .yml to .yaml for consistency

This reduces unnecessary CI runs on PRs that don't modify nix files.
Copilot AI review requested due to automatic review settings December 9, 2025 16:35
@ryoppippi ryoppippi requested a review from a team as a code owner December 9, 2025 16:35
@pkg-pr-new

pkg-pr-new Bot commented Dec 9, 2025

Copy link
Copy Markdown

Open in StackBlitz

npm i https://pkg.pr.new/StackOneHQ/stackone-ai-node/@stackone/ai@190

commit: c0a5227

Copilot AI left a comment

Copy link
Copy Markdown
Contributor

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 optimizes the nix-flake CI workflow by adding path filters to the pull_request trigger and fixing the workflow file extension reference for consistency.

Key Changes:

  • Added paths filter to pull_request trigger to match the existing push trigger configuration
  • Corrected workflow file extension reference from .yml to .yaml

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

@cubic-dev-ai cubic-dev-ai Bot left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

No issues found across 1 file

@ryoppippi ryoppippi enabled auto-merge (squash) December 9, 2025 17:45

@glebedel glebedel left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM

@ryoppippi ryoppippi merged commit bb66cf0 into main Dec 9, 2025
10 checks passed
@ryoppippi ryoppippi deleted the ci/flake-check-paths-filter branch December 9, 2025 18:11
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.

3 participants