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
Support repository access token on Bitbucket Cloud #1481
Support repository access token on Bitbucket Cloud #1481
Conversation
@@ -105,6 +107,12 @@ def fetch_pr_from_branch(branch_name) | |||
end | |||
|
|||
def fetch_access_token(environment) | |||
access_token = environment["DANGER_BITBUCKETCLOUD_REPO_ACCESSTOKEN"] |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
It is the same environment variable as danger-js.
Most of the documentation comments in this repository has been written in Markdown.
@@ -10,10 +10,31 @@ module Danger | |||
# | |||
# ### Token Setup | |||
# | |||
# 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. |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
There was a problem hiding this comment.
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 👍🏻
Overview
Close #1463
Test
I added some specs and did integration test on https://bitbucket.org/manicmaniac/danger-examples/pull-requests/1