Skip to content

Support repository access token on Bitbucket Cloud#1481

Merged
manicmaniac merged 8 commits into
danger:masterfrom
manicmaniac:support-bitbucket-cloud-repo-access-token
Feb 12, 2024
Merged

Support repository access token on Bitbucket Cloud#1481
manicmaniac merged 8 commits into
danger:masterfrom
manicmaniac:support-bitbucket-cloud-repo-access-token

Conversation

@manicmaniac

@manicmaniac manicmaniac commented Feb 9, 2024

Copy link
Copy Markdown
Member

Overview

Close #1463

Test

I added some specs and did integration test on https://bitbucket.org/manicmaniac/danger-examples/pull-requests/1

@manicmaniac manicmaniac self-assigned this Feb 9, 2024
end

def fetch_access_token(environment)
access_token = environment["DANGER_BITBUCKETCLOUD_REPO_ACCESSTOKEN"]

@manicmaniac manicmaniac Feb 9, 2024

Copy link
Copy Markdown
Member Author

Choose a reason for hiding this comment

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

@manicmaniac manicmaniac marked this pull request as ready for review February 9, 2024 18:37
Most of the documentation comments in this repository has been written
in Markdown.
#
# Add `DANGER_BITBUCKETCLOUD_USERNAME` and `DANGER_BITBUCKETCLOUD_PASSWORD` to your pipeline repository variable
# or instead using `DANGER_BITBUCKETCLOUD_OAUTH_KEY` and `DANGER_BITBUCKETCLOUD_OAUTH_SECRET`.
# For username and password, you need to set.

Copy link
Copy Markdown
Member Author

Choose a reason for hiding this comment

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

@orta orta left a comment

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

Not an expert on bitbucket, but looks right based on danger js implementation 👍🏻

@manicmaniac manicmaniac merged commit 4f0de68 into danger:master Feb 12, 2024
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.

Bitbucket Cloud access token

2 participants