Gitea Service: Use single release page (>= 1.12) #47

Merged
jolheiser merged 3 commits from 6543/gitea_changelog_tool:gitea-service-change-tag-URL into master 2020-08-07 13:42:03 +00:00
Owner

close #42

close #42
6543 added this to the v0.2.0 milestone 2020-08-07 12:54:37 +00:00
6543 added the
enhancement
label 2020-08-07 12:54:37 +00:00
6543 added 1 commit 2020-08-07 12:54:39 +00:00
gitea >= 1.12 has single release page
Some checks failed
continuous-integration/drone/pr Build is failing
e12449228d
jaqra approved these changes 2020-08-07 13:04:53 +00:00
Dismissed
service/gitea.go Outdated
@ -38,6 +38,9 @@ func (ge *Gitea) Generate() (string, []Entry, error) {
}
tagURL := fmt.Sprintf("## [%s](%s/%s/%s/%s?q=&type=all&state=closed&milestone=%d) - %s", ge.Milestone, ge.BaseURL, ge.Owner, ge.Repo, from, milestoneID, time.Now().Format("2006-01-02"))
if err := client.CheckServerVersionConstraint(">=1.12"); err == nil {
Member

Should also check if changelog from pulls. I mean !(ge.Issues)

Should also check if changelog from pulls. I mean `!(ge.Issues)`
Member

Sorry. Missclick. did not want to approve

Sorry. Missclick. did not want to approve
6543 added 1 commit 2020-08-07 13:09:26 +00:00
try to fix lint
All checks were successful
continuous-integration/drone/pr Build is passing
a99ee8917e
Author
Owner

Should also check if changelog from pulls. I mean !(ge.Issues)

@jaqra and why?

> Should also check if changelog from pulls. I mean `!(ge.Issues)` @jaqra and why?
jolheiser reviewed 2020-08-07 13:30:38 +00:00
Dismissed
service/gitea.go Outdated
@ -87,0 +88,4 @@
if err := c.CheckServerVersionConstraint(">=1.12"); err == nil {
return fmt.Sprintf("## [%s](%s/%s/%s/releases/tag/%s) - %s", mileName, baseURL, owner, repo, mileName, time.Now().Format("2006-01-02"))
}
return fmt.Sprintf("## [%s](%s/%s/%s/%s?q=&type=all&state=closed&milestone=%d) - %s", mileName, baseURL, owner, repo, from, mileID, time.Now().Format("2006-01-02"))
Owner

I tend to want the happy path at the end of the func, but if this is how it's styled in other places that's fine.

I tend to want the happy path at the end of the func, but if this is how it's styled in other places that's fine.
Author
Owner

no problem will change it

no problem will change it
6543 marked this conversation as resolved
Member

Should also check if changelog from pulls. I mean !(ge.Issues)

@jaqra and why?

Ignore my comment :)

> > Should also check if changelog from pulls. I mean `!(ge.Issues)` > > @jaqra and why? Ignore my comment :)
6543 added 1 commit 2020-08-07 13:39:25 +00:00
code format: "...happy path at the end..."
All checks were successful
continuous-integration/drone/pr Build is passing
03b5bdf423
6543 requested review from jolheiser 2020-08-07 13:40:59 +00:00
jolheiser approved these changes 2020-08-07 13:41:25 +00:00
Dismissed
jolheiser merged commit b3574b331b into master 2020-08-07 13:42:01 +00:00
6543 deleted branch gitea-service-change-tag-URL 2020-08-07 13:42: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#47
No description provided.