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

Clean up repo #407

Merged

shahthepro merged 3 commits into master from shah/clean-up-repo

Apr 10, 2024

Collaborator

shahthepro commented Apr 10, 2024 •

edited

Loading

Runs forge fmt and adds "Contracts Linter" action to CI
Fixes forge config file
Remove all DApp related code and doc

If you made a contract change, make sure to complete the checklist below before merging it in master.

Contract change checklist:

Code reviewed by 2 reviewers.
Copy & paste code review security checklist below this checklist.
Unit tests pass
Slither tests pass with no warning
Echidna tests pass if PR includes changes to OUSD contract (not automated, run manually on local)


          Clean up repo

9d548a0

shahthepro requested review from DanielVF and sparrowDom

April 10, 2024 14:33

shahthepro added 2 commits

April 10, 2024 20:05


          Add linter for contracts

3c6770e


          Tweak CI params

808d6bb

shahthepro merged commit ec62dda into master

3 checks passed

shahthepro deleted the shah/clean-up-repo branch

April 10, 2024 15:40

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet