Skip to content

feat: update workflow to give labels #165

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Open
wants to merge 7 commits into
base: master
Choose a base branch
from

Conversation

0xZaddyy
Copy link

PR Description

Add automated PR labeling system

This PR introduces an automated labeling system for pull requests based on which crates are modified. The system will apply appropriate labels (e.g., C-client, C-node, C-types) to PRs automatically as they are opened or updated.

Changes included:

  1. Added manage-pr.yml workflow file that:

    • Triggers on PR events (opened, synchronized, reopened)
    • Checks out both the base branch and PR merge commit
    • Dynamically generates a label configuration
    • Applies appropriate labels using the GitHub labeler action
  2. Added gen-label.sh script in the contrib/ folder that:

    • Scans the repository for crates
    • Generates a label configuration file mapping paths to labels
    • Includes safety checks to prevent accidental overwrites

@tcharding
Copy link
Member

Thanks for working on this! One problem with this type of PR is we don't know if it works until after it is triggered. Would you consider merging this into master on your own tree then raising a PR against your tree and then we can see if this works?

@tcharding
Copy link
Member

Oh and since it looks like a bunch of this is from rust-bitcoin can you please put that in the commit log.

@0xZaddyy
Copy link
Author

Alright sir, i will do all these now.

@tcharding
Copy link
Member

Thanks man, hit me with a link when you have it all working.

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