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

Issue 927 security monitor #950

Closed
wants to merge 2,041 commits into from

Conversation

LurkyLunk
Copy link

Resolves #927 #927

rndquu and others added 30 commits October 6, 2023 21:13
linter: update eslint flag
ci: using foundry from 2 Oct 2023
* fix: match  contract type according to diamond architecture
* fix: removed protocolcontracts in dependencies to avoid infinite loop
refactor: update creditclock to facet
* refactor(deploy): get CreditClockFacet selectors dynamically
We need this deployed by New Year's Eve.
…828)

* chore: remove unused hardhat absolute path replacement

Resolves: #826
gitcoindev and others added 20 commits May 30, 2024 13:57
As requested during pull request review.
…ns empty array

Prevent fromJSON failure by checking provide_contracts output before
triggering matrix strategy.

Resolves: #942
chore: skip check_storage_layout check if provide_contracts job retur…
@molecula451
Copy link
Member

let me know if this isn't the best branch for merging to.

it is not, this is unnaceptable, fork and clean branch and work from there

@LurkyLunk LurkyLunk deleted the issue-927-security-monitor branch July 30, 2024 18:02
@LurkyLunk LurkyLunk restored the issue-927-security-monitor branch July 30, 2024 18:02
@LurkyLunk LurkyLunk deleted the issue-927-security-monitor branch July 30, 2024 18:02
@LurkyLunk LurkyLunk restored the issue-927-security-monitor branch July 30, 2024 18:07
@LurkyLunk
Copy link
Author

it isnt letting me merge to developement branch is there a specific branch i should be comparing too?

@molecula451
Copy link
Member

image

you have to fork the branch https://github.com/ubiquity/ubiquity-dollar/ and make changes there, then send PR

@LurkyLunk
Copy link
Author

oh ok i thought i was supposed to create a draft before making any commits

@molecula451
Copy link
Member

image

looks like you selected "main" branch which is no update, please select "development"

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.

6 participants