Skip to content
This repository has been archived by the owner on Apr 29, 2024. It is now read-only.

celestia: import celestia-node directly #6

Merged
merged 7 commits into from
Oct 16, 2023
Merged

celestia: import celestia-node directly #6

merged 7 commits into from
Oct 16, 2023

Conversation

tuxcanfly
Copy link
Collaborator

@tuxcanfly tuxcanfly commented Oct 11, 2023 •
edited
Loading

Overview

This PR imports celestia-node directly and removes the dependency on celestia-openrpc. Fixes #3

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

@tuxcanfly tuxcanfly marked this pull request as ready for review October 16, 2023 21:50
@Manav-Aggarwal
Copy link
Member

Can we update celestia_da_test.go as well?

nashqueue
nashqueue approved these changes Oct 16, 2023
View reviewed changes
Copy link
Member

@nashqueue nashqueue 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 merged commit aff6f46 into main Oct 16, 2023
1 check passed
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Reviewers

@Manav-Aggarwal Manav-Aggarwal Manav-Aggarwal approved these changes

@nashqueue nashqueue nashqueue approved these changes

@tzdybal tzdybal Awaiting requested review from tzdybal

Assignees

@tuxcanfly tuxcanfly

Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.

[Feature Request]: Remove Celestia OpenRPC dependecy.
3 participants
@tuxcanfly @Manav-Aggarwal @nashqueue