Skip to content

Conversation

@SimonFair
Copy link
Contributor

@SimonFair SimonFair commented Jan 23, 2026

Reverts #2529

Summary by CodeRabbit

  • Chores
    • Refined build workflow to focus on relevant files, resulting in optimized package contents and improved build efficiency.

✏️ Tip: You can customize this high-level summary in your review settings.

@coderabbitai
Copy link
Contributor

coderabbitai bot commented Jan 23, 2026

Caution

Review failed

The pull request is closed.

Walkthrough

This pull request modifies the GitHub Actions workflow to narrow PR trigger conditions by removing three directory paths from the filter, update the logging message to specify emhttp files, and restrict tarball packaging to only the usr/ directory instead of the entire build tree.

Changes

Cohort / File(s) Summary
Workflow PR trigger and packaging
\.github/workflows/pr-plugin-build\.yml
Narrowed PR trigger by removing etc/**, sbin/**, and share/** paths; updated "No files changed" log message to "No emhttp files changed"; restricted tarball creation to only include usr/ directory instead of entire build tree

Estimated code review effort

🎯 1 (Trivial) | ⏱️ ~5 minutes

Poem

🐰 A workflow so lean, paths narrowed just right,
Only usr/ in tarballs, packaging tight!
Less noise in the triggers, clearer logs shine,
The build becomes swift—configuration divine! ✨


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 and usage tips.

@SimonFair SimonFair merged commit 11f0619 into master Jan 23, 2026
3 of 4 checks passed
@SimonFair SimonFair deleted the revert-2529-Update-Worker-for-plugin-build- branch January 23, 2026 15:50
@github-actions
Copy link

🧹 PR Test Plugin Cleaned Up

The test plugin and associated files for this PR have been removed from the preview environment.


🤖 This comment is automatically generated when a PR is closed.

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