Update Structs #486

Merged
6543 merged 18 commits from 6543/go-sdk:update-structs into master 2021-02-13 19:24:01 +00:00
Owner
  • FileOptions: add Signoff
  • Commit: add CommitAffectedFiles
  • BranchProtection, CreateBranchProtectionOption, EditBranchProtectionOption: add BlockOnOfficialReviewRequests
  • MigrateRepoOption: add MirrorInterval & enable gogs as source option
  • EditRepoOption: Add new fields
  • Repository: Add new fields & related structs
  • PullReview: Add fields
  • StopWatch: Add new fields
  • EditIssueOption: Add option to delete Deadline
  • EditUserOption: lot of options got optional

close #479

* FileOptions: add Signoff * Commit: add CommitAffectedFiles * BranchProtection, CreateBranchProtectionOption, EditBranchProtectionOption: add BlockOnOfficialReviewRequests * MigrateRepoOption: add MirrorInterval & enable gogs as source option * EditRepoOption: Add new fields * Repository: Add new fields & related structs * PullReview: Add fields * StopWatch: Add new fields * EditIssueOption: Add option to delete Deadline * EditUserOption: lot of options got optional close #479
6543 added this to the v0.14.0 milestone 2021-02-12 22:05:14 +00:00
6543 added 1 commit 2021-02-12 22:05:14 +00:00
6543 added the
kind/enhancement
kind/breaking
labels 2021-02-12 22:06:42 +00:00
Author
Owner

-> EditUserOption: lot of options got optional breaking!

-> EditUserOption: lot of options got optional **breaking!**
6543 force-pushed update-structs from 9747aff8b8 to 62bf840fae 2021-02-13 05:33:34 +00:00 Compare
6543 added 1 commit 2021-02-13 05:40:16 +00:00
Add migration hints for breaking change
Some checks failed
continuous-integration/drone/pr Build is failing
7417c4b8e2
6543 added 1 commit 2021-02-13 16:14:43 +00:00
Merge branch 'master' into update-structs
All checks were successful
continuous-integration/drone/pr Build is passing
f73d506d8b
6543 added 1 commit 2021-02-13 16:19:02 +00:00
impruve docs
All checks were successful
continuous-integration/drone/pr Build is passing
fb8a671da2
lunny approved these changes 2021-02-13 16:29:27 +00:00
Dismissed
zeripath reviewed 2021-02-13 16:37:43 +00:00
Dismissed
@ -27,1 +28,4 @@
- [#467 Remove & Rename TrackedTimes list functions](https://gitea.com/gitea/go-sdk/pulls/467)
## Some Fields of Structs got optional
Owner

New Optional Fields

New Optional Fields
6543 marked this conversation as resolved
zeripath reviewed 2021-02-13 16:39:17 +00:00
Dismissed
@ -28,0 +33,4 @@
The EditUserOption struct is only affected currently.
For example Email type changed from `string` to `*string`.
The easyest migration path is, to war your options with:
Owner

s/easyest/easiest

I don't understand to war your options with?

s/easyest/easiest I don't understand `to war your options with`?
6543 marked this conversation as resolved
zeripath reviewed 2021-02-13 16:41:14 +00:00
Dismissed
@ -80,1 +79,4 @@
}
case GitServiceGogs:
if len(opt.AuthToken) == 0 {
return fmt.Errorf("gogs require token authentication")
Owner

gogs requires token authentication

gogs require**s** token authentication
6543 marked this conversation as resolved
6543 added 1 commit 2021-02-13 16:46:24 +00:00
X requires token authentication
All checks were successful
continuous-integration/drone/pr Build is passing
72914776ae
6543 added 1 commit 2021-02-13 16:52:42 +00:00
New Optional Fields
All checks were successful
continuous-integration/drone/pr Build is passing
578a56b7eb
zeripath reviewed 2021-02-13 17:47:24 +00:00
Dismissed
@ -28,0 +30,4 @@
## New Optional Fields
`EditUserOption` struct is has gained several new Optional fields.
Owner

The EditUserOption struct has gained several new Optional fields.

The `EditUserOption` struct has gained several new Optional fields.
6543 marked this conversation as resolved
6543 added 1 commit 2021-02-13 18:54:56 +00:00
next
Some checks failed
continuous-integration/drone/pr Build is failing
2d5e3f4a51
zeripath approved these changes 2021-02-13 18:57:08 +00:00
Dismissed
zeripath left a comment
Owner

Apart from the double period at the end of the line. LGTM.

Apart from the double period at the end of the line. LGTM.
@ -28,0 +30,4 @@
## New Optional Fields
The `EditUserOption` struct has gained several new Optional fields..
Owner

oops double terminal period here

oops double terminal period here
6543 marked this conversation as resolved
6543 added 1 commit 2021-02-13 19:00:02 +00:00
next-try
All checks were successful
continuous-integration/drone/pr Build is passing
48ff1a903a
6543 merged commit 8947cd3b00 into master 2021-02-13 19:24:00 +00:00
6543 referenced this issue from a commit 2021-02-13 19:24:00 +00:00
6543 deleted branch update-structs 2021-02-13 19:24:10 +00:00
Sign in to join this conversation.
No description provided.