Skip to content

Conversation

matiasedgeandnode
Copy link
Contributor

I didn't add test to solhint because that lit up with +300 warnings. Probably best left for another PR.

@matiasedgeandnode
Copy link
Contributor Author

Should we do this for all lint:sol scripts?

Copy link

Add prettier to test files

Generated at commit: 3ad5707aa55514c06ba7c2939912b0db8b2bdc6d

🚨 Report Summary

Severity Level Results
Contracts Critical
High
Medium
Low
Note
Total
2
4
0
15
39
60
Dependencies Critical
High
Medium
Low
Note
Total
0
0
0
0
0
0

For more details view the full report in OpenZeppelin Code Inspector

Copy link
Member

@tmigone tmigone 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, i think we can address other changes as needed.

lit up with +300 warnings

yikes

@tmigone tmigone merged commit 0df54e3 into horizon Feb 18, 2025
2 of 5 checks passed
@tmigone tmigone deleted the ma/horizon-fix-prettier branch February 18, 2025 13:44
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants