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

Update formatting given recent forge fmt changes #220

Merged
merged 3 commits into from
May 16, 2024
Merged

Conversation

ermyas
Copy link
Contributor

@ermyas ermyas commented May 14, 2024

Foundry doesn't currently have a stable release, only nightly versions.
Since the last commit to this codebase, Foundry's formatter has received updates.
This means that the formatting output on the codebase is different in the latest version of Foundry.
This PR aims to update the formatting in the codebase in-line with Foundry's latest formatter version.

@ermyas ermyas requested a review from drinkcoffee May 14, 2024 03:08
@ermyas ermyas requested review from a team as code owners May 14, 2024 03:08
Copy link

openzeppelin-code bot commented May 14, 2024

Update formatting given recent forge fmt changes

Generated at commit: bd0fc564c8cde365bb3cc369c3006f23ef46fcb5

🚨 Report Summary

Severity Level Results
Contracts Critical
High
Medium
Low
Note
Total
2
1
0
11
27
41
Dependencies Critical
High
Medium
Low
Note
Total
0
0
0
0
0
0

For more details view the full report in OpenZeppelin Code Inspector

drinkcoffee
drinkcoffee previously approved these changes May 14, 2024
Copy link
Contributor

@drinkcoffee drinkcoffee left a comment

Choose a reason for hiding this comment

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

Looks good to me

Benjimmutable
Benjimmutable previously approved these changes May 14, 2024
jasonzwli
jasonzwli previously approved these changes May 14, 2024
@drinkcoffee drinkcoffee enabled auto-merge (squash) May 15, 2024 22:36
@drinkcoffee drinkcoffee disabled auto-merge May 15, 2024 22:36
@drinkcoffee drinkcoffee dismissed stale reviews from jasonzwli, Benjimmutable, and themself via bd0fc56 May 15, 2024 22:37
@drinkcoffee drinkcoffee merged commit 24dc024 into main May 16, 2024
6 of 7 checks passed
@Benjimmutable Benjimmutable deleted the fix/formatting branch May 16, 2024 23:16
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Development

Successfully merging this pull request may close these issues.

4 participants