Added mvn repository support #22
Conversation
There was a problem hiding this comment.
Oh, this is great! Thanks for the PR!
There's a few parts of dead code that can be removed (pointed out in the review comments), and I'd also love to have an entry in CHANGELOG.md to let people know that the new version now has Maven support.
Can you also make the change to -help output ("mvn" value for -l) in README.md Usage - section?
I'll do a bit of testing still, but this looks really good!
mvn.go
Outdated
| "time" | ||
| ) | ||
|
|
||
| type MvnResponse struct { |
There was a problem hiding this comment.
This struct can be removed, as pointed out by the linter.
mvn.go
Outdated
| } `json:"time"` | ||
| } | ||
|
|
||
| type MvnResponseUnpublished struct { |
There was a problem hiding this comment.
This struct can be removed, as pointed out by the linter.
|
Glad to help; fixed al problems and changed CHANGELOG.md & README.md First GO project I ever worked on; never to old to learn. |
No description provided.