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

Release 0.1.0-alpha.2 #14

Merged
merged 6 commits into from
Jan 27, 2024
Merged

Release 0.1.0-alpha.2 #14

merged 6 commits into from
Jan 27, 2024

Conversation

mzabaluev
Copy link
Contributor

No description provided.

Need to satisfy the limit imposed by crates.io.
"network-programming" is a better category for cometbft-proto than
"database".
"protobuf" is a more precise keyword.
@mzabaluev mzabaluev added the chore Changes not related to functional areas or features label Jan 26, 2024
Copy link

@andynog andynog left a comment

Choose a reason for hiding this comment

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

LGTM, a couple of comments in regards license

Copy link

Choose a reason for hiding this comment

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

I know this is a test, but should it include a license field ?

Copy link
Contributor Author

Choose a reason for hiding this comment

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

These are helper crates that are not meant to be published. I can throw in the license just as well.

Copy link

Choose a reason for hiding this comment

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

same comment as above regarding the license

Remove the version field from Cargo.toml of most of these crates as well.
These are optional for crates that don't get published in registries.
@mzabaluev mzabaluev merged commit 03974f5 into main Jan 27, 2024
24 checks passed
@mzabaluev mzabaluev deleted the release/v0.1.0-alpha.2 branch January 27, 2024 09:20
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