aiforge

10792 Commits

197 Branches

346 MB

814 Download

Tree: 3ebf1d1781

Commit Graph

Author	SHA1	Message	Date
Marge Bouvier	8149ed7040	Issue with Migration rule v111 (#9449 ) * Issue with Migration rule v111 I was facing some issues with migration rule v111 * Update v111.go Fix pgsql test cases Co-authored-by: Antoine GIRARD <sapk@users.noreply.github.com>	6 years ago
David Svantesson	bac4b78e09	Branch protection: Possibility to not use whitelist but allow anyone with write access (#9055 ) * Possibility to not use whitelist but allow anyone with write access * fix existing test * rename migration function * Try to give a better name for migration step * Clear settings if higher level setting is not set * Move official reviews to db instead of counting approvals each time * migration * fix * fix migration * fix migration * Remove NOT NULL from EnableWhitelist as migration isn't possible * Fix migration, reviews are connected to issues. * Fix SQL query issues in GetReviewersByPullID. * Simplify function GetReviewersByIssueID * Handle reviewers that has been deleted * Ensure reviews for test is in a well defined order * Only clear and set official reviews when it is an approve or reject.	6 years ago

Author

SHA1

Message

Date

Marge Bouvier

8149ed7040

Issue with Migration rule v111 (#9449 )

* Issue with Migration rule v111

I was facing some issues with migration rule v111

* Update v111.go

Fix pgsql test cases

Co-authored-by: Antoine GIRARD <sapk@users.noreply.github.com>

6 years ago

David Svantesson

bac4b78e09

Branch protection: Possibility to not use whitelist but allow anyone with write access (#9055 )

* Possibility to not use whitelist but allow anyone with write access

* fix existing test

* rename migration function

* Try to give a better name for migration step

* Clear settings if higher level setting is not set

* Move official reviews to db instead of counting approvals each time

* migration

* fix

* fix migration

* fix migration

* Remove NOT NULL from EnableWhitelist as migration isn't possible

* Fix migration, reviews are connected to issues.

* Fix SQL query issues in GetReviewersByPullID.

* Simplify function GetReviewersByIssueID

* Handle reviewers that has been deleted

* Ensure reviews for test is in a well defined order

* Only clear and set official reviews when it is an approve or reject.

6 years ago

2 Commits (3ebf1d178151a953be5eb56402c093e90bcfff13)