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

Remove/archive the Light Node #879

Closed
thanethomson opened this issue May 13, 2021 · 0 comments · Fixed by #881
Closed

Remove/archive the Light Node #879

thanethomson opened this issue May 13, 2021 · 0 comments · Fixed by #881
Assignees
Labels
enhancement New feature or request light-node

Comments

@thanethomson
Copy link
Contributor

thanethomson commented May 13, 2021

As per our discussion in the tendermint-rs call yesterday, if nobody's using it directly we should strongly consider removing the light-node crate from the repository.

The reasons for this are:

  1. The code to facilitate a Light Node is generally used as part of another higher-level library or application (e.g. ibc-rs), and isn't generally directly instantiated by itself (except perhaps for testing purposes).
  2. We should eventually consider positioning the tendermint-rs repo as a framework/collection of libraries, as opposed to mixing in binaries. Binaries should possibly be separated out at a later date as issues relating to them will often be operational and tangential to framework-level concerns.
  3. We want to minimize the amount of code we need to maintain in this repo.

What's the definition of "done" for this issue?

When the light-node crate's been deleted from this repo.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
enhancement New feature or request light-node
Projects
None yet
Development

Successfully merging a pull request may close this issue.

1 participant