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): remove forge debug subcommand #9606

Merged
merged 2 commits into from
Jan 6, 2025

Conversation

9547
Copy link
Contributor

@9547 9547 commented Dec 31, 2024

Motivation

Closes #9523

@zerosnacks ptal

Solution

@yash-atreya yash-atreya added the T-likely-breaking Type: requires changes that can be breaking label Jan 2, 2025
@zerosnacks zerosnacks changed the title chore(forge): rm subcommand debug chore(forge): remove forge debug subcommand Jan 2, 2025
Copy link
Collaborator

@grandizzy grandizzy left a comment

Choose a reason for hiding this comment

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

makes sense!

@zerosnacks
Copy link
Member

zerosnacks commented Jan 6, 2025

Related PR for the book, removing forge debug: foundry-rs/book#1382
Proposed changes lgtm and are complete

@zerosnacks zerosnacks merged commit dc8d980 into foundry-rs:master Jan 6, 2025
22 checks passed
@9547 9547 deleted the rm-forge-debug branch January 6, 2025 22:45
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
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 redundant standalone debug command from forge
4 participants