Skip to content

Conversation

@castrojo
Copy link
Contributor

@castrojo castrojo commented Jan 7, 2026

Summary

Skip expensive container builds when only documentation files are changed, reducing CI time and GitHub Actions usage.

Problem

Currently, every change triggers a full container build (~5-10 minutes), even when only editing markdown documentation files. This wastes CI resources and delays doc-only PR merges.

Solution

Add paths-ignore filters to the build workflow to skip builds when only these files change:

  • *.md - All markdown files (README.md, AGENTS.md, etc.)
  • LICENSE - License file
  • .gitignore - Git configuration

What Still Triggers Builds ✓

  • Containerfile changes
  • system_files/** changes
  • Justfile or **.just changes
  • bluefin-branding submodule updates
  • Workflow file changes
  • Manual triggers (workflow_dispatch)
  • Merge queue triggers

Benefits

  • ✅ Faster PR merges for documentation
  • ✅ Reduced GitHub Actions minutes
  • ✅ No unnecessary container image pushes
  • ✅ Follows same pattern as validate-just.yml
  • ✅ Consistent with industry best practices

Testing

This PR itself only changes the workflow file, so it will trigger a build (as expected). Future documentation PRs will skip the build.

To verify:

  1. Merge this PR
  2. Create a test PR that only modifies README.md
  3. Observe that the build workflow is skipped

References

Add paths-ignore filters to build workflow to skip expensive container
builds when only documentation files are modified. This reduces CI time
and GitHub Actions usage for documentation-only PRs.

Changes:
- Skip builds when only *.md files change (README, AGENTS, etc.)
- Skip builds for LICENSE and .gitignore changes
- Manual triggers (workflow_dispatch) still work
- Merge queue builds still work
- All code changes still trigger builds

Benefits:
- Faster PR merges for docs
- Reduced GitHub Actions minutes
- No unnecessary container image pushes
- Follows same pattern as validate-just.yml

Assisted-by: Claude 3.5 Sonnet via GitHub Copilot
@gemini-code-assist
Copy link

Note

Gemini is unable to generate a review for this pull request due to the file types involved not being currently supported.

@dosubot dosubot bot added the size:XS This PR changes 0-9 lines, ignoring generated files. label Jan 7, 2026
@dosubot
Copy link

dosubot bot commented Jan 7, 2026

Related Documentation

Checked 8 published document(s) in 0 knowledge base(s). No updates required.

How did I do? Any feedback?  Join Discord

@castrojo castrojo enabled auto-merge January 7, 2026 00:42
@dosubot dosubot bot added the lgtm This PR has been approved by a maintainer label Jan 7, 2026
Copy link
Member

@tulilirockz tulilirockz left a comment

Choose a reason for hiding this comment

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

This will also skip the MOTD stuff but whatever

@castrojo castrojo added this pull request to the merge queue Jan 11, 2026
Merged via the queue into main with commit bf330e0 Jan 11, 2026
2 checks passed
@castrojo castrojo deleted the ci/skip-builds-for-docs branch January 11, 2026 00:24
@castrojo castrojo added the kind/github-action Anything having to do with GHA and automation label Jan 11, 2026
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

kind/github-action Anything having to do with GHA and automation lgtm This PR has been approved by a maintainer size:XS This PR changes 0-9 lines, ignoring generated files.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants