-
Notifications
You must be signed in to change notification settings - Fork 11
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
Add dYdX commits to cosmos v0.47.1 #11
Conversation
* Fix lint/gocritic error * Disable some workflow jobs
// empty/reset the deliver state | ||
app.deliverState = nil | ||
|
||
if app.commiter != nil { | ||
app.commiter(app.checkState.ctx) | ||
} | ||
|
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Change potentially affects state.
Call sequence:
(*github.com/cosmos/cosmos-sdk/baseapp.BaseApp).Commit (baseapp/abci.go:452)
You have successfully added a new CodeQL configuration |
@jonfung-dydx your pull request is missing a changelog! |
3955a45
to
f1c6c95
Compare
Would you be able to squash the last 3 commits into 1 so that it's easier to layer commits next time? |
5. Cherry-pick each dydx-created commit from the current default branch, in order, on to the new `dydx-fork-$VERSION` branch (note: you may want to consider creating multiple PRs for this process if there are difficulties or merge conflicts). For example, `git cherry-pick <commit hash>`. You can verify the first commit by seeing the most recent commit sha for the `$VERSION` (i.e `v0.47.0-alpha2`) tag on the `cosmos/cosmos-sdk` repo, and taking the next commit from that sha. | ||
6. Open a PR to merge the second branch (`totoro/dydxCommits`) into the first (`dydx-fork-v0.47.0-alpha2`). Get approval, and merge. | ||
7. Update `dydxprotocol/v4` by following the steps in "Making Changes to the fork" above. | ||
8. Set `dydx-fork-$VERSION` as the [default branch](https://docs.github.com/en/repositories/configuring-branches-and-merges-in-your-repository/managing-branches-in-your-repository/changing-the-default-branch) in this repository. |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Should we add a note about the GitHub workflow permissioning and an example of those errors? It requires a GitHub admin to whitelist any new workflows.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Done
c58e5c4
to
0a6795c
Compare
Following instructions in readme to layer our commits on top of cosmos v0.47.1