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.

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

Better document reentrancy_guard() #3893

Closed
wants to merge 1 commit into from
Closed

Better document reentrancy_guard() #3893

wants to merge 1 commit into from

Conversation

nfurfaro
Copy link
Contributor

@nfurfaro nfurfaro commented Jan 25, 2023 •
edited
Loading

@nfurfaro nfurfaro requested a review from otrho as a code owner January 25, 2023 20:41
@nfurfaro nfurfaro self-assigned this Jan 25, 2023
@adlerjohn adlerjohn added the lib: std Standard library label Jan 25, 2023
@otrho otrho requested a review from adlerjohn January 26, 2023 01:18
@mohammadfawaz
Copy link
Contributor

@jtriley-eth is in the process of moving the guard to sway-libs in FuelLabs/sway-libs#83. Should we just make this same change there instead?

@nfurfaro nfurfaro mentioned this pull request Jan 26, 2023
Merged
@nfurfaro
Copy link
Contributor Author

@mohammadfawaz yes, these changes should be made in the other repo as you suggested. Closing this one :)

@nfurfaro nfurfaro closed this Jan 26, 2023
@nfurfaro nfurfaro deleted the furnic/rename-reentrancy-guard branch February 2, 2023 22:11
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@otrho otrho Awaiting requested review from otrho

@adlerjohn adlerjohn Awaiting requested review from adlerjohn

Assignees

@nfurfaro nfurfaro

Labels
documentation lib: std Standard library
Projects
Fuel Network
Status: Done
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.

Rename reentrancy guard
3 participants
@nfurfaro @mohammadfawaz @adlerjohn