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

List user permission for repo #63

Merged
merged 6 commits into from
May 17, 2017

Conversation

j0nathan33
Copy link
Contributor

No description provided.

@cdancy cdancy self-assigned this May 11, 2017
@cdancy cdancy added this to the v0.0.16 milestone May 11, 2017
@cdancy
Copy link
Owner

cdancy commented May 16, 2017

LGTM, and all tests pass, just need to resolve conflicts.

# Conflicts:
#	src/main/java/com/cdancy/bitbucket/rest/features/RepositoryApi.java
#	src/test/java/com/cdancy/bitbucket/rest/features/RepositoryApiLiveTest.java
#	src/test/java/com/cdancy/bitbucket/rest/features/RepositoryApiMockTest.java
@cdancy
Copy link
Owner

cdancy commented May 17, 2017

Looks like we need to resolve conflicts first. I'll wait until that is done, and this is closed, before proceeding with reviewing the next PR's.

# Conflicts:
#	src/main/java/com/cdancy/bitbucket/rest/features/RepositoryApi.java
#	src/test/java/com/cdancy/bitbucket/rest/features/RepositoryApiMockTest.java
@cdancy cdancy requested review from cdancy and removed request for cdancy May 17, 2017 19:06
@cdancy
Copy link
Owner

cdancy commented May 17, 2017

LGTM and all tests pass as expected. Thanks again @j0nathan33 !

@cdancy cdancy merged commit 53862ff into cdancy:master May 17, 2017
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.

2 participants