Skip to content
This repository was archived by the owner on Jan 22, 2025. It is now read-only.

Follow up to persistent tower with tests and API cleaning (bp #12350)#12972

Merged
mergify[bot] merged 1 commit intov1.4from
mergify/bp/v1.4/pr-12350
Oct 19, 2020
Merged

Follow up to persistent tower with tests and API cleaning (bp #12350)#12972
mergify[bot] merged 1 commit intov1.4from
mergify/bp/v1.4/pr-12350

Conversation

@mergify
Copy link
Copy Markdown
Contributor

@mergify mergify Bot commented Oct 19, 2020

This is an automated backport of pull request #12350 done by Mergify

Mergify commands and options

More conditions and actions can be found in the documentation.

You can also trigger Mergify actions by commenting on this pull request:

  • @Mergifyio refresh will re-evaluate the rules
  • @Mergifyio rebase will rebase this PR on its base branch
  • @Mergifyio update will merge the base branch into this PR
  • @Mergifyio backport <destination> will backport this PR on <destination> branch

Additionally, on Mergify dashboard you can:

  • look at your merge queues
  • generate the Mergify configuration with the config editor.

Finally, you can contact us on https://mergify.io/

* Follow up to persistent tower

* Ignore for now...

* Hard-code validator identities for easy reasoning

* Add a test for opt. conf violation without tower

* Fix compile with rust < 1.47

* Remove unused method

* More move of assert tweak to the asser pr

* Add comments

* Clean up

* Clean the test addressing various review comments

* Clean up a bit

(cherry picked from commit 54517ea)
@mergify mergify Bot added the automerge Merge this Pull Request automatically once CI passes label Oct 19, 2020
@codecov
Copy link
Copy Markdown

codecov Bot commented Oct 19, 2020

Codecov Report

Merging #12972 into v1.4 will decrease coverage by 0.0%.
The diff coverage is 78.5%.

@@           Coverage Diff            @@
##            v1.4   #12972     +/-   ##
========================================
- Coverage   81.9%    81.9%   -0.1%     
========================================
  Files        361      361             
  Lines      85097    85112     +15     
========================================
+ Hits       69729    69733      +4     
- Misses     15368    15379     +11     

@mergify mergify Bot merged commit 8a526f2 into v1.4 Oct 19, 2020
@mergify mergify Bot deleted the mergify/bp/v1.4/pr-12350 branch October 19, 2020 08:48
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Labels

automerge Merge this Pull Request automatically once CI passes

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant