Skip to content

Fix: expand wallet regex parsing for #6645#6683

Closed
darlina-bounty-codex wants to merge 1 commit into
Scottcjn:mainfrom
darlina-bounty-codex:issue-6645-regex
Closed

Fix: expand wallet regex parsing for #6645#6683
darlina-bounty-codex wants to merge 1 commit into
Scottcjn:mainfrom
darlina-bounty-codex:issue-6645-regex

Conversation

@darlina-bounty-codex
Copy link
Copy Markdown
Contributor

Resolves #6645

Summary

This PR expands the regular expression used to parse wallet addresses in the \�ward_rtc.py\ script.

Changes Made

  • Updated _WALLET_RE\ to support the requested variations:
    • \Payout wallet:`n - \Payout address:`n - \RTC wallet:`n - \miner_id:`n - \miner_id=`n - \wallet=`n -
      tc_wallet:`n- Changed the separator matching from just :\ to [:=]\ to allow for =\ assignments.

This provides more flexibility in PR descriptions and reduces the need for manual interventions for bounties.

wallet: RTC1410e82d545ce0b3ffd21ca83e2465a8f2c3a64e

@github-actions
Copy link
Copy Markdown
Contributor

Welcome to RustChain! Thanks for your first pull request.

Before we review, please make sure:

  • Non-doc PRs have a BCOS-L1 or BCOS-L2 label
  • Doc-only PRs are exempt from BCOS tier labels when they only touch docs/**, *.md, or common image/PDF files
  • New code files include an SPDX license header
  • You've tested your changes against the live node

Bounty tiers: Micro (1-10 RTC) | Standard (20-50) | Major (75-100) | Critical (100-150)

A maintainer will review your PR soon. Thanks for contributing!

@github-actions github-actions Bot added BCOS-L1 Beacon Certified Open Source tier BCOS-L1 (required for non-doc PRs) ci size/XS PR: 1-10 lines labels May 31, 2026
@Scottcjn
Copy link
Copy Markdown
Owner

Superseded by #6676 (keon0711), which lands the same wallet-directive parsing for #6645 — filed ~6.5h earlier and far more complete (adds payout wallet / payout address / rtc wallet / miner_id directives + an RTC-address fallback + unit tests, vs the 3-line regex here). First-poster-per-issue, so #6676 is the one we'll merge.

Closing as superseded — no reflection on the fix, it's just the same change arriving second. Thanks for the contribution. 🔥

@Scottcjn Scottcjn closed this May 31, 2026
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

BCOS-L1 Beacon Certified Open Source tier BCOS-L1 (required for non-doc PRs) ci size/XS PR: 1-10 lines

Projects

None yet

Development

Successfully merging this pull request may close these issues.

[TRACKING] Auto-pay workflow should parse 'Payout address:' / 'Payout wallet:' / 'RTC wallet:' variants from PR body

2 participants