Skip to content

Implement and test GitHub authentication actions#53

Merged
txsmith merged 1 commit intoisApproved-v2from
github-auth-actions
Jun 14, 2017
Merged

Implement and test GitHub authentication actions#53
txsmith merged 1 commit intoisApproved-v2from
github-auth-actions

Conversation

@txsmith
Copy link
Collaborator

@txsmith txsmith commented Jun 14, 2017

No description provided.

@preview-code
Copy link

preview-code bot commented Jun 14, 2017

This pull request can be reviewed with Preview Code.
To speed up the review process and get better feedback on your changes, please order your changes.

@txsmith txsmith force-pushed the github-auth-actions branch from 648d913 to f378d8f Compare June 14, 2017 09:55
@txsmith txsmith force-pushed the github-auth-actions branch from f378d8f to 27a9344 Compare June 14, 2017 10:09
@txsmith txsmith merged commit 87991d2 into isApproved-v2 Jun 14, 2017
@txsmith txsmith deleted the github-auth-actions branch June 14, 2017 10:10
eanker added a commit that referenced this pull request Jun 29, 2017
* Added initial version of approvingAPI v2

* Allow users to set the approval status multiple times

* Implement getApprovals

* Implement and test GitHub authentication actions (#53)

* Add tests for getApprovals actions

* Added two tests (then stepper was empty)

* Test that evaluation of getApproval finishes

* Added enpoint for fetching hunk approvals

* Added tests for getHunkApprovals

* Added javadoc for approvalsAPI

* Change all occurrences of Seq to List

* Implement DeleteGroup in DatabaseInterpreter

* Ensure ApproveStatus is never null

* Implement FetchHunksForGroup in DatabaseInterpreter

* Make Approvals refer to a hunk by their ID

* Constrain hunks to be unique per pull request

* Coerce the jOOQ api into handling 'INSERT INTO ... RETURNING ...' queries

* Change the database dsl to return entire hunks/approvals

* Fix tests for database service

* Re-implement hunk and approval fetching

* Implement generic deserialization of types with one field

* Implement wrapped type serialization

* Test wrapped type de/serialization

* Implement github authentication actions and a caching interpreter

* Import hunks when a PR is opened (#54)
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant