-
Notifications
You must be signed in to change notification settings - Fork 8
Rename vss-client crate to vss-client-ng
#46
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
Conversation
|
👋 Thanks for assigning @tnull as a reviewer! |
9d3715a to
d8086d0
Compare
tnull
left a comment
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
LGTM, some questions.
| @@ -1,8 +1,8 @@ | |||
| [package] | |||
| name = "vss-client" | |||
| name = "vss-client-ng" | |||
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
While we're here, mind tacking-on a second commit adding an authors field so people can see who are the primary contacts for the crate? I propose:
authors = ["Leo Nash <[email protected]>", "Elias Rohrer <[email protected]>"]
Cargo.toml
Outdated
| license = "MIT OR Apache-2.0" | ||
| edition = "2021" | ||
| homepage = "https://lightningdevkit.org/" | ||
| repository = "https://github.com/lightningdevkit/vss-rust-client" |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Should we reflect the change/repo rename to rust-vss-client here?
| @@ -1,8 +1,8 @@ | |||
| [package] | |||
| name = "vss-client" | |||
| name = "vss-client-ng" | |||
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Might also want to make this change in the header of README.md, which should probably be named after the crate, not the repo.
Use the new crate name in `README.md`, not the repo name. Update the repository link in preparation for repo move.
d8086d0 to
36ac919
Compare
No description provided.