Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Add possibility to delete Default Reviewers #84

Closed
wants to merge 79 commits into from

Conversation

j0nathan33
Copy link
Contributor

My last PR for next few week :)

@cdancy cdancy modified the milestones: v0.9.0, v0.0.16 Jun 6, 2017
Jonathan Doire added 3 commits June 6, 2017 15:13
…o defaultReviewers-update

# Conflicts:
#	src/main/java/com/cdancy/bitbucket/rest/features/DefaultReviewersApi.java
#	src/test/java/com/cdancy/bitbucket/rest/BaseBitbucketApiLiveTest.java
#	src/test/java/com/cdancy/bitbucket/rest/features/DefaultReviewersApiLiveTest.java
#	src/test/java/com/cdancy/bitbucket/rest/features/DefaultReviewersApiMockTest.java
cdancy added a commit that referenced this pull request Jun 6, 2017
cdancy added a commit that referenced this pull request Jun 6, 2017
* First commit of default reviewers

* Rename Conditions to condition

* Add mock test

* Add live Test

* Fix build

* Add more test for branchApiLiveTest

* Add Fallback for Condition

* Update Object Condition for add ErrorsHolder

* Add rest api for create Condition

* Edit MatcherId to add new value

* Add live test

* Add mock Test

* Fix junit

* Fix build

* Add endpoint for Condition update

* Add liveTest

* Add mock Test

* Fix build

* Add endpoint for delete Condition

* Add live Test

* Fix build

* Fix build

* Add mock Test

* Add annotation Documentation for listConditions

* Add annotation Documentation for createCondition

* Add annotation Documentation for updateCondition

* Add annotation Documentation for deleteCondition

* Add missing annotation

* Fix build

* Refactore test after change from master branch

* Refactore test after change from master branch

* fix build

* Fix build

* Fix build

* Fix build

* Edit code after codeReview from another PR (List defaultReviewers)

* Fix code after codeReview

* Fix issues #89

* Rename test class

* Add field to get conditionId for next Step

* Add live test to get listConditions

* Change HardCode User to defaultUser

* Fix build

* Fix build

* Edit code after codeReviews from another PR

* Edit code after codeReviews from another PR

* Edit code after codeReview

* Remove test dependency don't need anymore

* Edit rest named for deleteCondition

* Fixes for PR-#84
@cdancy
Copy link
Owner

cdancy commented Jun 6, 2017

Closing as this has been merged as part of #92

@cdancy cdancy closed this Jun 6, 2017
@cdancy
Copy link
Owner

cdancy commented Jun 6, 2017

Thanks again @j0nathan33 !

@j0nathan33 j0nathan33 deleted the defaultReviewers-delete branch June 6, 2017 21:01
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

2 participants