Use Predefind Versions & Compare Function #442

Merged
zeripath merged 11 commits from 6543/go-sdk:bugfix_440_usePredefind-ConstrainChecks into master 2020-11-12 20:59:05 +00:00
Owner

close #440 and speedup this lib

close #440 and speedup this lib
6543 added this to the v0.14.0 milestone 2020-11-08 23:23:28 +00:00
6543 added the
kind/bug
need/backport
labels 2020-11-08 23:23:28 +00:00
6543 force-pushed bugfix_440_usePredefind-ConstrainChecks from 39b2538167 to a58f932127 2020-11-09 00:54:23 +00:00 Compare
6543 added the
status/needs-reviews
label 2020-11-09 00:54:43 +00:00
lunny reviewed 2020-11-09 01:01:16 +00:00
Dismissed
@ -45,0 +53,4 @@
func (c *Client) checkServerVersionGreaterThanOrEqual(v *version.Version) error {
c.versionLock.RLock()
if c.serverVersion == nil {
c.versionLock.RUnlock()
Owner

Is this really locked?

Is this really locked?
Author
Owner

yes (from two lines above)

yes (from two lines above)
Owner
https://gitea.com/gitea/go-sdk/pulls/456
lunny marked this conversation as resolved
6543 added
has/backport
and removed
need/backport
labels 2020-11-09 01:06:07 +00:00
Author
Owner

Backport: #446

Backport: #446
6543 added 1 commit 2020-11-09 01:09:24 +00:00
Merge branch 'master' into bugfix_440_usePredefind-ConstrainChecks
Some checks failed
continuous-integration/drone/pr Build is failing
61b14fbedd
6543 added 1 commit 2020-11-09 01:14:39 +00:00
update test data
All checks were successful
continuous-integration/drone/pr Build is passing
eb8e2b2c97
noerw approved these changes 2020-11-09 15:41:06 +00:00
Dismissed
noerw left a comment
Member

Can we have _ separators in the version constants to reduce ambiguity?
Otherwise lgtm

Can we have `_` separators in the version constants to reduce ambiguity? Otherwise lgtm
6543 removed the
status/needs-reviews
label 2020-11-09 16:21:42 +00:00
6543 added 1 commit 2020-11-09 16:24:40 +00:00
version use underscore
All checks were successful
continuous-integration/drone/pr Build is passing
f55758ce8b
6543 added 1 commit 2020-11-09 16:25:35 +00:00
version use underscore
All checks were successful
continuous-integration/drone/pr Build is passing
df7f5c7845
6543 requested review from lunny 2020-11-09 17:19:38 +00:00
6543 added 1 commit 2020-11-09 20:55:29 +00:00
Merge branch 'master' into bugfix_440_usePredefind-ConstrainChecks
Some checks failed
continuous-integration/drone/pr Build is failing
6e151fba82
6543 added 1 commit 2020-11-09 21:19:26 +00:00
Update TestData
All checks were successful
continuous-integration/drone/pr Build is passing
964dee2260
zeripath approved these changes 2020-11-12 20:58:30 +00:00
Dismissed
zeripath merged commit 9c81fa936f into master 2020-11-12 20:59:04 +00:00
6543 deleted branch bugfix_440_usePredefind-ConstrainChecks 2020-11-12 20:59:37 +00:00
Sign in to join this conversation.
No description provided.