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

chore(forge): rm regex for --debug and --decode-internal #9572

Merged
merged 4 commits into from
Jan 8, 2025

Conversation

yash-atreya
Copy link
Member

Motivation

Closes #8931

Solution

Remove the regex arg for --debug and --decode-internal. Use bool instead as the matching test is derived from --mt

@yash-atreya yash-atreya added the T-likely-breaking Type: requires changes that can be breaking label Dec 18, 2024
@yash-atreya yash-atreya changed the title chore(forge): rm regex for --debug and --decode-internal [DO NOT MERGE] chore(forge): rm regex for --debug and --decode-internal Dec 18, 2024
@yash-atreya yash-atreya marked this pull request as ready for review December 18, 2024 10:03
@yash-atreya yash-atreya added C-forge Command: forge Cmd-forge-test Command: forge test labels Dec 19, 2024
@yash-atreya yash-atreya changed the title [DO NOT MERGE] chore(forge): rm regex for --debug and --decode-internal chore(forge): rm regex for --debug and --decode-internal Jan 7, 2025
Copy link
Member

@zerosnacks zerosnacks left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

lgtm

@grandizzy grandizzy merged commit ad09bbe into master Jan 8, 2025
22 checks passed
@grandizzy grandizzy deleted the yash/fix-8931 branch January 8, 2025 06:11
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
C-forge Command: forge Cmd-forge-test Command: forge test T-likely-breaking Type: requires changes that can be breaking
Projects
Status: No status
Development

Successfully merging this pull request may close these issues.

chore: remove regex argument of forge test --debug and --decode-internal
4 participants