Skip to content

Allow usage of private golang repositories#500

Merged
imjasonh merged 4 commits into
GoogleCloudPlatform:masterfrom
mlesar:go-git-config
Jun 11, 2019
Merged

Allow usage of private golang repositories#500
imjasonh merged 4 commits into
GoogleCloudPlatform:masterfrom
mlesar:go-git-config

Conversation

@mlesar
Copy link
Copy Markdown
Contributor

@mlesar mlesar commented Jun 10, 2019

Added GIT_CONFIG_VAL environment variable with which you can set for example GitHub access token so that you can get the code from private repositories without using vendor dir.
Example:

  env:
    - 'GO111MODULE=on'
    - 'GIT_CONFIG_VAL=url."https://${_GITHUB_ACCESS_TOKEN}@github.com/".insteadOf "https://github.com/"'

@googlebot
Copy link
Copy Markdown

Thanks for your pull request. It looks like this may be your first contribution to a Google open source project (if not, look below for help). Before we can look at your pull request, you'll need to sign a Contributor License Agreement (CLA).

📝 Please visit https://cla.developers.google.com/ to sign.

Once you've signed (or fixed any issues), please reply here (e.g. I signed it!) and we'll verify it.


What to do if you already signed the CLA

Individual signers
Corporate signers

ℹ️ Googlers: Go here for more info.

@googlebot googlebot added the cla: no CLA does not look good :( label Jun 10, 2019
@mlesar
Copy link
Copy Markdown
Contributor Author

mlesar commented Jun 10, 2019

I signed it!

@googlebot
Copy link
Copy Markdown

CLAs look good, thanks!

ℹ️ Googlers: Go here for more info.

@googlebot googlebot added cla: yes CLA looks good and removed cla: no CLA does not look good :( labels Jun 10, 2019
@codrienne codrienne requested a review from imjasonh June 11, 2019 17:42
@imjasonh imjasonh merged commit a2cb38d into GoogleCloudPlatform:master Jun 11, 2019
jakebiesinger-onduo pushed a commit to jakebiesinger/cloud-builders that referenced this pull request Nov 3, 2019
* Set git config value

* Fix spacing

* Cleaning up

* New line at the end
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

cla: yes CLA looks good

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants