Skip to content
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

Add Timed Bounty Parsing and Fix Repository Input #3932

Open
wants to merge 1 commit into
base: main
Choose a base branch
from

Conversation

vidipsingh
Copy link
Contributor

@vidipsingh vidipsingh commented Mar 13, 2025

User description

User description

  • This PR updates the assign-issues.yml workflow to support the timed bounty feature requested in issue Have the ability to set a timed bounty on an issue #3883.
  • It adds a script to parse bounty-amount and bounty-hours from the issue body and includes the required repository input for the BLT-Action.

Fixes: #3883


PR Type

enhancement, configuration changes


Description

  • Added support for timed bounty in assign-issues.yml.

  • Introduced parsing of bounty-amount and bounty-hours from issue body.

  • Updated BLT-Action configuration with new inputs.


Changes walkthrough 📝

Relevant files
Configuration changes
assign-issues.yml
Add timed bounty parsing to workflow                                         

.github/workflows/assign-issues.yml

  • Added bounty-amount input to parse bounty value.
  • Added bounty-hours input to set bounty duration.
  • Updated BLT-Action step with new inputs.
  • +2/-0     

    Need help?
  • Type /help how to ... in the comments thread for any questions about PR-Agent usage.
  • Check out the documentation for more information.
  • Copy link
    Contributor

    PR Reviewer Guide 🔍

    Here are some key observations to aid the review process:

    🎫 Ticket compliance analysis 🔶

    3883 - Partially compliant

    Compliant requirements:

    • Add a variable for hours to support timed bounties.

    Non-compliant requirements:

    []

    Requires further human verification:

    []

    ⏱️ Estimated effort to review: 2 🔵🔵⚪⚪⚪
    🧪 No relevant tests
    🔒 No security concerns identified
    ⚡ Recommended focus areas for review

    Possible Issue

    The default values for bounty-amount and bounty-hours may not handle invalid or missing data in github.event.issue.body robustly. Ensure proper validation and fallback mechanisms are in place.

    bounty-amount: ${{ github.event.issue.body.bounty || 0 }}
    bounty-hours: ${{ github.event.issue.body.hours || 24 }}

    Copy link
    Contributor

    PR Code Suggestions ✨

    @DonnieBLT
    Copy link
    Collaborator

    Can you show the screenshots of it working please?

    @vidipsingh
    Copy link
    Contributor Author

    Can you show the screenshots of it working please?

    Hi @DonnieBLT, could you please guide me on how to simulate this action?

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

    Successfully merging this pull request may close these issues.

    Have the ability to set a timed bounty on an issue
    2 participants