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

extsvc/bitbucketserver: Implement User listing in Client #4352

Merged
merged 10 commits into from Jun 5, 2019

Conversation

tsenart
Copy link
Contributor

@tsenart tsenart commented Jun 4, 2019

This commit implements user listing in the bitbucketserver.Client
which is needed as part of #1108.

API documentation: https://docs.atlassian.com/bitbucket-server/rest/6.1.2/bitbucket-rest.html#idp363

Test plan: go test

This commit implements user listing in the `bitbucketserver.Client`
which is needed as part of #1108.
@tsenart tsenart added this to the 3.5 milestone Jun 4, 2019
@codecov
Copy link

codecov bot commented Jun 4, 2019

Codecov Report

Merging #4352 into master will increase coverage by 0.02%.
The diff coverage is 65.78%.

Impacted Files Coverage Δ
pkg/extsvc/bitbucketserver/client.go 50.31% <65.78%> (ø)

pkg/extsvc/bitbucketserver/client.go Outdated Show resolved Hide resolved
Copy link
Contributor

@mrnugget mrnugget left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Very, very nice! I really like how you refactored the Client 👏

I left some comments/questions, no blockers though

pkg/extsvc/bitbucketserver/client.go Show resolved Hide resolved
pkg/extsvc/bitbucketserver/client_test.go Outdated Show resolved Hide resolved
pkg/extsvc/bitbucketserver/client_test.go Outdated Show resolved Hide resolved
pkg/extsvc/bitbucketserver/client.go Show resolved Hide resolved
pkg/extsvc/bitbucketserver/client.go Outdated Show resolved Hide resolved
pkg/extsvc/bitbucketserver/client.go Show resolved Hide resolved
pkg/extsvc/bitbucketserver/client.go Outdated Show resolved Hide resolved
pkg/extsvc/bitbucketserver/client.go Outdated Show resolved Hide resolved
pkg/extsvc/bitbucketserver/client_test.go Outdated Show resolved Hide resolved
pkg/extsvc/bitbucketserver/client_test.go Show resolved Hide resolved
pkg/extsvc/bitbucketserver/client_test.go Outdated Show resolved Hide resolved
@tsenart
Copy link
Contributor Author

tsenart commented Jun 5, 2019

@mrnugget, @keegancsmith: Please take another look. Addressed most of your feedback.

@mrnugget
Copy link
Contributor

mrnugget commented Jun 5, 2019

@mrnugget, @keegancsmith: Please take another look. Addressed most of your feedback.

LGTM!

@keegancsmith
Copy link
Member

LGTM

@tsenart tsenart merged commit dc37d86 into master Jun 5, 2019
@tsenart tsenart deleted the core/bitbucket-user-listing branch June 5, 2019 16:14
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.

None yet

4 participants