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

Rename the crates to cometbft*, remove deprecated crates #4

Merged
merged 13 commits into from
Dec 11, 2023

Conversation

mzabaluev
Copy link
Contributor

To publish the crates as part of cometbft-rs, we need to rename them.

These crates have not been actively developed and are considered deprecated, so it's better remove them at this point rather than maintain them as part of the new family:

  • tendermint-abci
  • tendermint-p2p
  • tendermint-std-ext
  • tendermint-test

All crates published as part of the forked cometbft-rs project
will use the cometbft* naming.
The tendermint-abci crate was deprecated and has a better alternative
maintained elsewhere, so it is removed rather than renamed.
Bulk search-and-replace changes.
Do not continue the  p2p crate as part of cometbft-rs, before the need for it
is clear. Also drop the auxiliary crates whose only use was to provide
functionality for, or test, tendermint-p2p.
@mzabaluev mzabaluev added the chore Changes not related to functional areas or features label Dec 11, 2023
@mzabaluev mzabaluev requested a review from a team December 11, 2023 10:36
@mzabaluev
Copy link
Contributor Author

An ICE in nightly 👀

Copy link
Contributor

@thanethomson thanethomson left a comment

Choose a reason for hiding this comment

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

Nice work! 🎉

@thanethomson
Copy link
Contributor

Does updating grcov to v0.8.19 perhaps fix the nightly coverage workflow failure?

@mzabaluev
Copy link
Contributor Author

Does updating grcov to v0.8.19 perhaps fix the nightly coverage workflow failure?

This is not grcov, the crash is in the compiler with the instrumentation flag turned on.

@thanethomson
Copy link
Contributor

Awesome! Glad to see we can execute the code coverage workflow using the stable toolchain. We'll be able to upload the coverage results to codecov.io when we make the repo public.

@thanethomson
Copy link
Contributor

thanethomson commented Dec 11, 2023

@mzabaluev will you be updating the root docs (e.g. the README, contributing guidelines) in this PR or a subsequent one?

I will file a separate PR, there were a lot of bulk changes, so I have only committed some opportunistic edits so far alongside the crate renaming.

@mzabaluev mzabaluev merged commit 7e1787c into main Dec 11, 2023
22 checks passed
@mzabaluev mzabaluev deleted the mikhail/rename-to-cometbft branch December 11, 2023 14:56
@mzabaluev mzabaluev mentioned this pull request Dec 12, 2023
2 tasks
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
chore Changes not related to functional areas or features
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants