Skip to content

feat!: require PR label for processing#7

Merged
dseeker merged 8 commits into
mainfrom
feat/pr-label-filtering
May 10, 2025
Merged

feat!: require PR label for processing#7
dseeker merged 8 commits into
mainfrom
feat/pr-label-filtering

Conversation

@dseeker

@dseeker dseeker commented May 10, 2025

Copy link
Copy Markdown
Contributor

Description

This PR adds a new feature that requires pull requests to have a specific label (default: "claudecoder") to be processed by the ClaudeCoder action. This improves control over which PRs get processed by the action.

Breaking Changes

This is a breaking change that will impact existing workflows:

  • PRs now require a label (default: 'claudecoder') to be processed by the action
  • Existing workflows will need to be updated to add labels to PRs that should be processed
  • Workflow triggers should be updated to include the 'labeled' event type for optimal performance

Implementation Details

  • Added a new required-label input parameter to the action (defaults to "claudecoder")
  • Updated the main logic to check for the required label before processing a PR
  • Added appropriate messages to inform users when PRs are skipped due to missing labels
  • Updated documentation across README, docs/index.md, and CHANGELOG
  • Fixed a bug in package.json where the major version script was incorrectly set to "minor"

Documentation Updates

  • Updated README.md with information about the label requirement and workflow configuration
  • Updated docs/index.md with consistent information
  • Added breaking change information to CHANGELOG.md
  • Updated example workflows to include the 'labeled' event type
  • Added configuration examples with custom label options

Testing

  • Tested with PRs with and without the required label to verify behavior
  • Ensured the early exit logic works correctly when label is missing
  • Confirmed appropriate comments are added to PRs without labels

This change will bump the version to 2.0.0 due to the breaking nature of the change.

@github-actions

Copy link
Copy Markdown

This PR was not processed by Claude 3.7 Sonnet because it doesn't have the required 'claudecoder' label. Add this label if you want AI assistance.

@github-actions

Copy link
Copy Markdown

✅ ClaudeCoder self-test complete! This PR has been processed by the ClaudeCoder action itself.

@github-actions

Copy link
Copy Markdown

This PR was not processed by Claude 3.7 Sonnet because it doesn't have the required 'claudecoder' label. Add this label if you want AI assistance.

@github-actions

Copy link
Copy Markdown

✅ ClaudeCoder self-test complete! This PR has been processed by the ClaudeCoder action itself.

@github-actions

Copy link
Copy Markdown

This PR was not processed by Claude 3.7 Sonnet because it doesn't have the required 'claudecoder' label. Add this label if you want AI assistance.

@github-actions

Copy link
Copy Markdown

✅ ClaudeCoder self-test complete! This PR has been processed by the ClaudeCoder action itself.

@dseeker dseeker merged commit b2c3eaa into main May 10, 2025
3 checks passed
@dseeker dseeker deleted the feat/pr-label-filtering branch May 10, 2025 09:32
@github-actions

Copy link
Copy Markdown

🎉 This PR is included in version 1.4.0 🎉

The release is available on GitHub release

Your semantic-release bot 📦🚀

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

Labels

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant