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

mv gov sub-module from shell into gov crate #3718

Merged
merged 4 commits into from
Sep 6, 2024
Merged

Conversation

tzemanovic
Copy link
Member

Describe your changes

Checklist before merging

  • If this PR has some consensus breaking changes, I added the corresponding breaking:: labels
    • This will require 2 reviewers to approve the changes

@tzemanovic tzemanovic force-pushed the tomas/gov-refactor branch 2 times, most recently from d3ada83 to bb64df3 Compare August 30, 2024 10:30
Copy link

codecov bot commented Aug 30, 2024

Codecov Report

Attention: Patch coverage is 82.91874% with 103 lines in your changes missing coverage. Please review.

Project coverage is 72.47%. Comparing base (ef00f27) to head (96d8e23).
Report is 24 commits behind head on main.

Files with missing lines Patch % Lines
crates/governance/src/finalize_block.rs 81.86% 78 Missing ⚠️
crates/node/src/shell/finalize_block.rs 72.97% 20 Missing ⚠️
crates/node/src/protocol.rs 0.00% 3 Missing ⚠️
crates/trans_token/src/lib.rs 95.45% 2 Missing ⚠️
Additional details and impacted files
@@            Coverage Diff             @@
##             main    #3718      +/-   ##
==========================================
+ Coverage   72.41%   72.47%   +0.05%     
==========================================
  Files         335      334       -1     
  Lines      104005   104127     +122     
==========================================
+ Hits        75320    75464     +144     
+ Misses      28685    28663      -22     

☔ View full report in Codecov by Sentry.
📢 Have feedback on the report? Share it here.

tzemanovic added a commit that referenced this pull request Aug 30, 2024
@tzemanovic tzemanovic marked this pull request as ready for review August 30, 2024 11:51
@tzemanovic tzemanovic mentioned this pull request Aug 30, 2024
1 task
@Fraccaman
Copy link
Member

another nice refactor, thanks @tzemanovic! We merged #3601, you need to rebase on main to fix e2e test that is failing!

@tzemanovic
Copy link
Member Author

tzemanovic commented Aug 30, 2024

I've added one more refactor to get rid of the tx and vp cache being passed through in 1e319fb

tzemanovic added a commit that referenced this pull request Aug 30, 2024
tzemanovic added a commit that referenced this pull request Aug 30, 2024
@Fraccaman Fraccaman self-requested a review September 2, 2024 08:04
@Fraccaman Fraccaman added the merge Ready to merge - mergifyio bot will add the PR to merge queue when all checks pass label Sep 2, 2024
mergify bot added a commit that referenced this pull request Sep 2, 2024
Copy link
Contributor

mergify bot commented Sep 2, 2024

Hey @tzemanovic, your pull request has been dequeued due to the following reason: CHECKS_FAILED.
Sorry about that, but you can requeue the PR by using @mergifyio requeue if you think this was a mistake.

@tzemanovic
Copy link
Member Author

@Mergifyio requeue

Copy link
Contributor

mergify bot commented Sep 2, 2024

requeue

✅ The queue state of this pull request has been cleaned. It can be re-embarked automatically

mergify bot added a commit that referenced this pull request Sep 2, 2024
mergify bot added a commit that referenced this pull request Sep 2, 2024
Copy link
Contributor

mergify bot commented Sep 2, 2024

Hey @tzemanovic, your pull request has been dequeued due to the following reason: CHECKS_FAILED.
Sorry about that, but you can requeue the PR by using @mergifyio requeue if you think this was a mistake.

tzemanovic added a commit that referenced this pull request Sep 2, 2024
@tzemanovic
Copy link
Member Author

@Mergifyio requeue

Copy link
Contributor

mergify bot commented Sep 2, 2024

requeue

❌ This pull request head commit has not been previously disembarked from queue.

@tzemanovic
Copy link
Member Author

the integration::ledger_tests::pgf_governance_proposal keeps failing in CI but locally it passes for me :/

@brentstone
Copy link
Collaborator

@tzemanovic I was having the same problem with that failing test in #3691 - at some point after a rebasing on main it passed but I never figured out why it was failing in CI

@tzemanovic
Copy link
Member Author

this one started failing after rebase - same in #3735

@tzemanovic tzemanovic mentioned this pull request Sep 3, 2024
1 task
tzemanovic added a commit that referenced this pull request Sep 3, 2024
tzemanovic added a commit that referenced this pull request Sep 4, 2024
mergify bot added a commit that referenced this pull request Sep 6, 2024
mergify bot added a commit that referenced this pull request Sep 6, 2024
mergify bot added a commit that referenced this pull request Sep 6, 2024
@mergify mergify bot merged commit 5e3418c into main Sep 6, 2024
23 checks passed
@mergify mergify bot deleted the tomas/gov-refactor branch September 6, 2024 10:23
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
merge Ready to merge - mergifyio bot will add the PR to merge queue when all checks pass
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants