-
Notifications
You must be signed in to change notification settings - Fork 489
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
feat(hub): enable repository_url
#3961
Conversation
5f97f4a
to
5663c81
Compare
pub description: String, | ||
pub license: String, | ||
pub manifest: Vec<String>, // Files in package, package-meta is implied, signature is omitted | ||
pub repository_url: Option<Url>, | ||
pub repository_commit_sha: Option<String>, |
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.
Lets leave out repository_commit_sha for now. We can re-introduce it later as a tag if needed.
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.
SGTM! Here is the change: 9babcd7
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 (just the nit of the sha field comment)
repository_url
and repository_commit_sha
repository_url
* feat(hub): enable `repository_url` and `repository_commit_sha` * fix: fmt issues * fix: rm commit sha
No description provided.