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

ci: Test with CometBFT v0.38.2 #210

Open
wants to merge 4 commits into
base: main
Choose a base branch
from

Conversation

thanethomson
Copy link
Member

@thanethomson thanethomson commented Nov 8, 2023

Ideally we'd be able to build/test/run this against multiple versions of Comet, but until such time that we have capacity and demand for that we should at least use the latest major release.

@thanethomson thanethomson added ci Continuous integration wip Work in progress labels Nov 8, 2023
Signed-off-by: Thane Thomson <connect@thanethomson.com>
Signed-off-by: Thane Thomson <connect@thanethomson.com>
Signed-off-by: Thane Thomson <connect@thanethomson.com>
@codecov-commenter
Copy link

codecov-commenter commented Nov 8, 2023

Codecov Report

Merging #210 (cd6fa2f) into main (b435f4d) will not change coverage.
Report is 2 commits behind head on main.
The diff coverage is n/a.

Additional details and impacted files
@@           Coverage Diff           @@
##             main     #210   +/-   ##
=======================================
  Coverage   61.00%   61.00%           
=======================================
  Files          18       18           
  Lines        2072     2072           
=======================================
  Hits         1264     1264           
  Misses        680      680           
  Partials      128      128           

Signed-off-by: Thane Thomson <connect@thanethomson.com>
@thanethomson thanethomson changed the title ci: Test with CometBFT v0.38.0 ci: Test with CometBFT v0.38.2 Nov 28, 2023
@@ -10,19 +10,21 @@ RUN apk update && \
apk upgrade && \
apk --no-cache add curl jq file shadow

RUN groupadd -g ${GID} tmgroup || true
Copy link
Member Author

Choose a reason for hiding this comment

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

Fixes a problem in CI.

@thanethomson thanethomson marked this pull request as ready for review November 28, 2023 11:48
@thanethomson thanethomson removed the wip Work in progress label Nov 28, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
ci Continuous integration
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

2 participants