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

lint and configure ci #254

Open
wants to merge 5 commits into
base: master
Choose a base branch
from
Open

lint and configure ci #254

wants to merge 5 commits into from

Conversation

faddat
Copy link

@faddat faddat commented Jan 19, 2024

Hi,

I've been following this work for a long time, and for a long time I've wanted to integrate it with Tendermint ....

But lately most call Tendermint CometBFT.

I've written this PR, which I visited this repo, to learn about getting addresses from kcp, but got distracted by my need to lint things, and I tbought this PR would be helpful.

cometbft/cometbft#1918

My plan here is to gain a better understanding of the kcp code by linting it some :).

Cheers!

-Jacob

The PR

  • lints / formats the code
  • removes broken or deprecated algorithms and replaces them with newer
  • adds CI

This was referenced Jan 21, 2024
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.

None yet

1 participant