8000 Allow inclusion of repository in the tag name, so ghcr and dockerub c… by lasarojc · Pull Request #345 · cometbft/cometbft · GitHub
[go: up one dir, main page]
More Web Proxy on the site http://driver.im/
Skip to content

Allow inclusion of repository in the tag name, so ghcr and dockerub c… #345

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

Merged
merged 9 commits into from
Feb 21, 2023

Conversation

lasarojc
Copy link
Contributor

…an be used, including in the same experiment

Should be ported to 0.37 and main.


PR checklist

  • Tests written/updated
  • Changelog entry added in .changelog (we use
    unclog to manage our changelog)
  • Updated relevant documentation (docs/ or spec/) and code comments

…an be used, including in the same experiment
@lasarojc lasarojc requested a review from a team as a code owner February 14, 2023 17:27
@lasarojc lasarojc self-assigned this Feb 14, 2023
@codecov
Copy link
codecov bot commented Feb 14, 2023

Codecov Report

Base: 48.40% // Head: 48.37% // Decreases project coverage by -0.04% ⚠️

Coverage data is based on head (25d7435) compared to base (38abebd).
Patch coverage: 0.00% of modified lines in pull request are covered.

❗ Current head 25d7435 differs from pull request most recent head 9431984. Consider uploading reports for the commit 9431984 to get more accurate results

Additional details and impacted files
@@             Coverage Diff             @@
##           v0.34.x     #345      +/-   ##
===========================================
- Coverage    48.40%   48.37%   -0.04%     
===========================================
  Files          280      280              
  Lines        49887    49908      +21     
===========================================
- Hits         24150    24145       -5     
- Misses       23878    23900      +22     
- Partials      1859     1863       +4     
Impacted Files Coverage Δ
test/e2e/generator/generate.go 9.18% <0.00%> (-0.71%) ⬇️
test/e2e/generator/main.go 0.00% <0.00%> (ø)
privval/signer_listener_endpoint.go 81.74% <0.00%> (-9.53%) ⬇️
privval/socket_listeners.go 79.72% <0.00%> (-4.06%) ⬇️
privval/signer_endpoint.go 74.72% <0.00%> (-2.20%) ⬇️
consensus/reactor.go 78.38% <0.00%> (-1.84%) ⬇️
statesync/syncer.go 75.07% <0.00%> (-0.58%) ⬇️
blockchain/v0/reactor.go 76.02% <0.00%> (-0.35%) ⬇️
consensus/state.go 70.30% <0.00%> (-0.25%) ⬇️
light/client.go 72.64% <0.00%> (-0.15%) ⬇️
... and 9 more

Help us with your feedback. Take ten seconds to tell us how you rate us. Have a feature suggestion? Share it here.

☔ View full report at Codecov.
📢 Do you have feedback about the report comment? Let us know in this issue.

Copy link
Contributor
@sergio-mena sergio-mena left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Thanks for this!

@sergio-mena sergio-mena merged commit 5bdaf56 into v0.34.x Feb 21, 2023
@sergio-mena sergio-mena deleted the lasarojc/registry-in-tag branch February 21, 2023 10:06
@sergio-mena sergio-mena added the backport-to-v0.37.x Tell Mergify to backport the PR to v0.37.x label Feb 21, 2023
lasarojc added a commit that referenced this pull request Feb 21, 2023
#345)

* Allow inclusion of repository in the tag name, so ghcr and dockerub can be used, including in the same experiment

* Moves UpgradeVersion to Node, so nodes can be upgraded to different versions. Uses multiversion to set UpgradeVersion.

* Updates the path for the local-version image.

* Do not use yoda conditions.

* Use the right code coverage definitions

* Revert use of . We will only do a maximum of 2 versions per run
lasarojc added a commit that referenced this pull request Feb 21, 2023
#345)

* Allow inclusion of repository in the tag name, so ghcr and dockerub can be used, including in the same experiment

* Moves UpgradeVersion to Node, so nodes can be upgraded to different versions. Uses multiversion to set UpgradeVersion.

* Updates the path for the local-version image.

* Do not use yoda conditions.

* Use the right code coverage definitions

* Revert use of . We will only do a maximum of 2 versions per run
lasarojc added a commit that referenced this pull request Feb 21, 2023
#345) (#362)

* Allow inclusion of repository in the tag name, so ghcr and dockerub can be used, including in the same experiment

* Moves UpgradeVersion to Node, so nodes can be upgraded to different versions. Uses multiversion to set UpgradeVersion.

* Updates the path for the local-version image.

* Do not use yoda conditions.

* Use the right code coverage definitions

* Revert use of . We will only do a maximum of 2 versions per run
mergify bot pushed a commit that referenced this pull request Feb 21, 2023
#345) (#362)

* Allow inclusion of repository in the tag name, so ghcr and dockerub can be used, including in the same experiment

* Moves UpgradeVersion to Node, so nodes can be upgraded to different versions. Uses multiversion to set UpgradeVersion.

* Updates the path for the local-version image.

* Do not use yoda conditions.

* Use the right code coverage definitions

* Revert use of . We will only do a maximum of 2 versions per run

(cherry picked from commit 45a08f3)
lasarojc added a commit that referenced this pull request Feb 21, 2023
#345) (#362) (#363)

* Allow inclusion of repository in the tag name, so ghcr and dockerub can be used, including in the same experiment

* Moves UpgradeVersion to Node, so nodes can be upgraded to different versions. Uses multiversion to set UpgradeVersion.

* Updates the path for the local-version image.

* Do not use yoda conditions.

* Use the right code coverage definitions

* Revert use of . We will only do a maximum of 2 versions per run

(cherry picked from commit 45a08f3)

Co-authored-by: Lasaro <lasaro@informal.systems>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
backport-to-v0.37.x Tell Mergify to backport the PR to v0.37.x
Projects
No open projects
Status: Done
Development

Successfully merging this pull request may close these issues.

2 participants
0