Skip to content

Trim GitHub Actions, tests, and/or hooks - #320

Merged
j9t merged 2 commits into
mainfrom
trim-actions
Aug 1, 2026
Merged

Trim GitHub Actions, tests, and/or hooks#320
j9t merged 2 commits into
mainfrom
trim-actions

Conversation

@j9t

@j9t j9t commented Aug 1, 2026

Copy link
Copy Markdown
Owner

Summary by CodeRabbit

  • Chores
    • Updated automated site publishing workflows to skip unnecessary runs for documentation, configuration, backtesting, and test-only changes.
    • Corrected the published site artifact location for more reliable deployments.
    • Improved continuous integration by running lint checks on the supported Linux configuration and retaining test coverage on macOS.
    • Streamlined workflow triggers and verification steps for more efficient builds.

Signed-off-by: Jens Oliver Meiert <jens@meiert.com>
@coderabbitai

coderabbitai Bot commented Aug 1, 2026

Copy link
Copy Markdown

Review Change Stack

No actionable comments were generated in the recent review. 🎉

ℹ️ Recent review info
⚙️ Run configuration

Configuration used: Organization UI

Review profile: ASSERTIVE

Plan: Pro Plus

Run ID: 084f27b8-a425-4746-aeef-ab41b0545687

📥 Commits

Reviewing files that changed from the base of the PR and between 244d803 and 6b2d0b0.

📒 Files selected for processing (2)
  • .github/workflows/github-pages.yml
  • .github/workflows/tests.yml

Walkthrough

The pull request updates two GitHub Actions workflows. The GitHub Pages workflow adds path filters and changes the artifact path. The test workflow limits linting to the Ubuntu LTS matrix job.

Changes

CI workflow updates

Layer / File(s) Summary
GitHub Pages trigger filtering
.github/workflows/github-pages.yml
The workflow ignores pushes limited to Markdown, .github/**, backtest/**, or tests/** files. It uploads the Pages artifact from demo/build.
Main test job linting
.github/workflows/tests.yml
The workflow uses main as the push branch filter. The main matrix job runs npm run lint only on Ubuntu with the LTS Node version. The macOS job retains npm test.

Estimated code review effort: 2 (Simple) | ~10 minutes

🚥 Pre-merge checks | ✅ 5
✅ Passed checks (5 passed)
Check name Status Explanation
Description Check ✅ Passed Check skipped - CodeRabbit’s high-level summary is enabled.
Title check ✅ Passed The title clearly describes the workflow and test reductions in the pull request.
Docstring Coverage ✅ Passed No functions found in the changed files to evaluate docstring coverage. Skipping docstring coverage check.
Linked Issues check ✅ Passed Check skipped because no linked issues were found for this pull request.
Out of Scope Changes check ✅ Passed Check skipped because no linked issues were found for this pull request.
✨ Finishing Touches
🧪 Generate unit tests (beta)
  • Create PR with unit tests
  • Commit unit tests in branch trim-actions

Thanks for using CodeRabbit! It's free for OSS, and your support helps us grow. If you like it, consider giving us a shout-out.

❤️ Share

Comment @coderabbitai help to get the list of available commands.

@coderabbitai coderabbitai Bot left a comment

Copy link
Copy Markdown

Choose a reason for hiding this comment

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

Actionable comments posted: 1

🤖 Prompt for all review comments with AI agents
Verify each finding against current code. Fix only still-valid issues, skip the
rest with a brief reason, keep changes minimal, and validate.

Inline comments:
In @.github/workflows/tests.yml:
- Around line 40-41: Update the workflow job containing the Lint step so npm run
lint executes only once rather than once per matrix entry. Move lint to a
standalone job or guard it to a single matrix combination, while preserving npm
test execution for every platform and Node.js matrix job.
🪄 Autofix (Beta)

Fix all unresolved CodeRabbit comments on this PR:

  • Push a commit to this branch (recommended)
  • Create a new PR with the fixes

ℹ️ Review info
⚙️ Run configuration

Configuration used: Organization UI

Review profile: ASSERTIVE

Plan: Pro Plus

Run ID: 804ffc68-0a96-4062-b99b-3314f325a01d

📥 Commits

Reviewing files that changed from the base of the PR and between 81a02af and 244d803.

📒 Files selected for processing (2)
  • .github/workflows/github-pages.yml
  • .github/workflows/tests.yml

Comment thread .github/workflows/tests.yml
Signed-off-by: Jens Oliver Meiert <jens@meiert.com>
@j9t
j9t merged commit 1863c81 into main Aug 1, 2026
10 checks passed
@j9t
j9t deleted the trim-actions branch August 1, 2026 14:05
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.

1 participant