Add flag to specify git tag #58

Merged
6543 merged 1 commits from 6543/gitea_changelog_tool:tag-flag into master 2022-03-13 02:33:03 +00:00
Owner

close #57

close #57
6543 added 1 commit 2022-03-09 01:34:20 +00:00
add flag to specify git tag
All checks were successful
continuous-integration/drone/pr Build is passing
30ae24bd56
zeripath approved these changes 2022-03-10 08:20:44 +00:00
noerw approved these changes 2022-03-10 10:42:41 +00:00
noerw left a comment
Member

just one nit

just one nit
@ -15,1 +15,3 @@
func New(serviceType, repo, baseURL, milestone, token string, issues bool) (Service, error) {
func New(serviceType, repo, baseURL, milestone, tag, token string, issues bool) (Service, error) {
if len(tag) == 0 {
tag = milestone
Member

could you add a comment explaining / warning that we default tag to milestone?

could you add a comment explaining / warning that we default tag to milestone?
Author
Owner

It's already in the flag description, should I print to stdout?

@noerw

It's already in the flag description, should I print to stdout? @noerw
Member

ah i see, never mind then.

ah i see, never mind then.
noerw marked this conversation as resolved
6543 merged commit f3ee0e5726 into master 2022-03-13 02:33:03 +00:00
6543 deleted branch tag-flag 2022-03-13 02:33:04 +00:00
6543 added this to the v0.3.0 milestone 2022-03-13 02:33:15 +00:00
6543 added the
feature
label 2022-03-13 02:33:21 +00:00
Sign in to join this conversation.
No reviewers
No Milestone
No Assignees
3 Participants
Notifications
Due Date
The due date is invalid or out of range. Please use the format 'yyyy-mm-dd'.

No due date set.

Dependencies

No dependencies set.

Reference: gitea/changelog#58
No description provided.