Skip to content

Make panic-backtrace required in validation #4011

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.

Sign up for GitHub

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

Jump to bottom
Open
MKowalski8 wants to merge 1 commit into
base: 3678-bump-minimal-scarb
Choose a base branch
from
Open

Make panic-backtrace required in validation #4011

MKowalski8 wants to merge 1 commit into from
+3 −40

Conversation

@MKowalski8
Copy link
Member

@MKowalski8 MKowalski8 commented Dec 10, 2025 •
edited
Loading

Closes #3679

Stack:

#4009
#4010
#4011
#4012

Introduced changes

  • Add panic-backtrace as required in check_profile function for backtrace

Checklist

  • Linked relevant issue
  • Updated relevant documentation
  • Added relevant tests
  • Performed self-review of the code
  • Added changes to CHANGELOG.md

@MKowalski8 MKowalski8 requested a review from a team as a code owner December 10, 2025 17:09
@MKowalski8 MKowalski8 requested review from franciszekjob and ksew1 and removed request for a team December 10, 2025 17:09
This was referenced Dec 10, 2025
Draft
Open
Open
@MKowalski8 MKowalski8 linked an issue Dec 10, 2025 that may be closed by this pull request
Make panic-backtrace required in validation #3679
Open
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

@cptartur cptartur cptartur approved these changes

@franciszekjob franciszekjob Awaiting requested review from franciszekjob franciszekjob is a code owner automatically assigned from foundry-rs/starknet-foundry

@ksew1 ksew1 Awaiting requested review from ksew1 ksew1 is a code owner automatically assigned from foundry-rs/starknet-foundry

Assignees

No one assigned

Labels

None yet

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

Make panic-backtrace required in validation

3 participants

@MKowalski8 @cptartur