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

Merge 1.2.0 back to compatible #9188

Merged
merged 29 commits into from
Jul 14, 2021
Merged

Conversation

mrmr1993
Copy link
Member

This PR contains the changes from 1.2.0, with merge conflicts resolved, with a few additional changes to fix the build.

lk86 and others added 29 commits April 29, 2021 13:58
improve-bad-balance-scripting Add conditional for the ----- lines
Upgrade ocaml-rs, tweak some bindings to be safer
[hotfix] Revert staged ledger diff changes
…-the-tip

Fix/super catchup stuck at the tip
…ad-release

`get_ledger` in sys thread for release
@mrmr1993 mrmr1993 added the ci-build-me Add this label to trigger a circle+buildkite build for this branch label Jul 12, 2021
@mrmr1993 mrmr1993 requested review from a team as code owners July 12, 2021 18:20
@mrmr1993
Copy link
Member Author

mrmr1993 commented Jul 13, 2021

Some of the constituent PRs in this still need review.

@lk86
Copy link
Contributor

lk86 commented Jul 13, 2021

The infra changes in #8815 and #9068 are both minor (unrelated to the daemon) and safe. The product changes in #9132 also lgtm. Added a check for those 3 where i have permission, github won't let me review the PR itself post-merge.

@mimoo
Copy link
Contributor

mimoo commented Jul 13, 2021

#8998 has no major changes, so LGTM from that point of view, still left some comments there.

@lk86 lk86 merged commit 95c66c8 into compatible Jul 14, 2021
@lk86 lk86 deleted the feature/merge-1.2.0-to-compatible branch July 14, 2021 03:18
@lk86
Copy link
Contributor

lk86 commented Jul 14, 2021

Merging to keep the two branches in sync, further work for #8998 can land as part of a future PR

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
ci-build-me Add this label to trigger a circle+buildkite build for this branch
Projects
None yet
Development

Successfully merging this pull request may close these issues.

6 participants