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

deps: use celestia-node v0.6.1, celestia-app v0.11.0 #38

Merged
merged 1 commit into from
Dec 19, 2022

Conversation

tzdybal
Copy link
Member

@tzdybal tzdybal commented Dec 16, 2022

Overview

Resolves: #37
Blocked-by: #40

Checklist

  • New and updated code has appropriate documentation
  • New and updated code has new and/or updated testing
  • Required CI checks are passing
  • Visual proof for any user facing features like CLI or documentation updates
  • Linked issues closed with keywords

@tzdybal tzdybal requested review from a team, tuxcanfly, Manav-Aggarwal and nashqueue and removed request for a team December 16, 2022 13:22
@tzdybal tzdybal self-assigned this Dec 16, 2022
@codecov-commenter
Copy link

Codecov Report

Merging #38 (26329b7) into main (966e9d4) will not change coverage.
The diff coverage is n/a.

@@           Coverage Diff           @@
##             main      #38   +/-   ##
=======================================
  Coverage   54.02%   54.02%           
=======================================
  Files           2        2           
  Lines          87       87           
=======================================
  Hits           47       47           
  Misses         34       34           
  Partials        6        6           

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

@@ -3,7 +3,7 @@ version: '3'
services:
core0:
container_name: core0
image: "ghcr.io/celestiaorg/celestia-app:v0.10.0-rc1"
image: "ghcr.io/celestiaorg/celestia-app:v0.10.0"
Copy link
Member

Choose a reason for hiding this comment

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

Regarding Issue #37 Should this be Version 10 or 11 ?

Copy link
Member

Choose a reason for hiding this comment

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

works with 0.10.0, error 13 with 0.11.0 , so I changed Title

Copy link
Member Author

Choose a reason for hiding this comment

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

We need v0.11.0 compatibility, as this is version used by Arabica testnet.

tuxcanfly
tuxcanfly previously approved these changes Dec 16, 2022
Copy link
Contributor

@tuxcanfly tuxcanfly left a comment

Choose a reason for hiding this comment

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

LGTM.

@nashqueue nashqueue changed the title deps: use celestia-node v0.6.1, celestia-app v0.11.0 deps: use celestia-node v0.6.1, celestia-app v0.10.0 Dec 16, 2022
nashqueue
nashqueue previously approved these changes Dec 16, 2022
@@ -3,7 +3,7 @@ version: '3'
services:
core0:
container_name: core0
image: "ghcr.io/celestiaorg/celestia-app:v0.10.0-rc1"
image: "ghcr.io/celestiaorg/celestia-app:v0.10.0"
Copy link
Member

Choose a reason for hiding this comment

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

works with 0.10.0, error 13 with 0.11.0 , so I changed Title

@tzdybal tzdybal dismissed stale reviews from nashqueue and tuxcanfly via 2e6cb04 December 16, 2022 22:56
@tzdybal tzdybal force-pushed the tzdybal/node_v0.6.1 branch from 26329b7 to 2e6cb04 Compare December 16, 2022 22:56
@tzdybal tzdybal changed the title deps: use celestia-node v0.6.1, celestia-app v0.10.0 deps: use celestia-node v0.6.1, celestia-app v0.11.0 Dec 16, 2022
@tzdybal tzdybal changed the base branch from main to tzdybal/fees December 16, 2022 22:57
Base automatically changed from tzdybal/fees to main December 19, 2022 13:50
@tzdybal tzdybal force-pushed the tzdybal/node_v0.6.1 branch from 2e6cb04 to 539d655 Compare December 19, 2022 13:53
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Update dependencies
5 participants