Skip to content
This repository has been archived by the owner on Jan 4, 2023. It is now read-only.

Add PanicReason #34

Merged
merged 4 commits into from
Dec 1, 2021
Merged

Add PanicReason #34

merged 4 commits into from
Dec 1, 2021

Conversation

vlopes11
Copy link
Contributor

@vlopes11 vlopes11 commented Dec 1, 2021

@vlopes11 vlopes11 self-assigned this Dec 1, 2021
@vlopes11 vlopes11 requested review from adlerjohn, Dentosal and rakita and removed request for Dentosal, adlerjohn and rakita December 1, 2021 12:08
@vlopes11 vlopes11 marked this pull request as draft December 1, 2021 12:09
src/panic_reason.rs Outdated Show resolved Hide resolved
@vlopes11 vlopes11 marked this pull request as ready for review December 1, 2021 12:59
vlopes11 added a commit to FuelLabs/fuel-tx that referenced this pull request Dec 1, 2021
vlopes11 added a commit to FuelLabs/fuel-tx that referenced this pull request Dec 1, 2021
Dentosal
Dentosal previously approved these changes Dec 1, 2021
Copy link
Member

@Dentosal Dentosal left a comment

Choose a reason for hiding this comment

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

LGTM

@vlopes11 vlopes11 merged commit 550e28f into master Dec 1, 2021
@vlopes11 vlopes11 deleted the vlopes11/panic-reason branch December 1, 2021 14:44
vlopes11 added a commit to FuelLabs/fuel-tx that referenced this pull request Dec 4, 2021
ControlCplusControlV pushed a commit to FuelLabs/fuel-vm that referenced this pull request Dec 1, 2022
xgreenx pushed a commit to FuelLabs/fuel-vm that referenced this pull request Dec 20, 2022
xgreenx pushed a commit to FuelLabs/fuel-vm that referenced this pull request Dec 20, 2022
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants