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

repos: add option include_all_branches to template repo request #1884

Merged

Conversation

zhouhaibing089
Copy link
Contributor

This option, when set to true, the created repository will get files
from all the branches, not just the default branch.

See create-a-repository-using-a-template for more information.

@google-cla
Copy link

google-cla bot commented Jun 7, 2021

We found a Contributor License Agreement for you (the sender of this pull request), but were unable to find agreements for all the commit author(s) or Co-authors. If you authored these, maybe you used a different email address in the git commits than was used to sign the CLA (login here to double check)? If these were authored by someone else, then they will need to sign a CLA as well, and confirm that they're okay with these being contributed to Google.
In order to pass this check, please resolve this problem and then comment @googlebot I fixed it.. If the bot doesn't comment, it means it doesn't think anything has changed.

ℹ️ Googlers: Go here for more info.

@google-cla google-cla bot added the cla: no label Jun 7, 2021
This option, when set to true, the created repository will get files
from all the branches, not just the default branch.

See [create-a-repository-using-a-template][1] for more information.

[1]: https://docs.github.com/en/rest/reference/repos#create-a-repository-using-a-template
@zhouhaibing089 zhouhaibing089 force-pushed the template-request-include-all-branches branch from bc96f22 to f72fa87 Compare June 7, 2021 17:59
@google-cla google-cla bot added cla: yes Indication that the PR author has signed a Google Contributor License Agreement. and removed cla: no labels Jun 7, 2021
@gmlewis
Copy link
Collaborator

gmlewis commented Jun 7, 2021

Thank you, @zhouhaibing089 !
Please run go generate ./... and then push (please do not force-push in this repo) the changes made to this PR in order to get the tests to pass.
See CONTRIBUTING.md for more information.

Run `go generate ./...`.
@zhouhaibing089
Copy link
Contributor Author

@gmlewis: Thanks for the tip. I now followed the guide.

@codecov
Copy link

codecov bot commented Jun 7, 2021

Codecov Report

Merging #1884 (d1f957d) into master (8751f3d) will not change coverage.
The diff coverage is n/a.

Impacted file tree graph

@@           Coverage Diff           @@
##           master    #1884   +/-   ##
=======================================
  Coverage   97.65%   97.65%           
=======================================
  Files         105      105           
  Lines        6783     6783           
=======================================
  Hits         6624     6624           
  Misses         86       86           
  Partials       73       73           
Impacted Files Coverage Δ
github/repos.go 98.70% <ø> (ø)

Continue to review full report at Codecov.

Legend - Click here to learn more
Δ = absolute <relative> (impact), ø = not affected, ? = missing data
Powered by Codecov. Last update 8751f3d...d1f957d. Read the comment docs.

Copy link
Collaborator

@gmlewis gmlewis left a comment

Choose a reason for hiding this comment

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

Thank you, @zhouhaibing089 !
LGTM.
Merging.

@gmlewis gmlewis merged commit 773936a into google:master Jun 7, 2021
@zhouhaibing089 zhouhaibing089 deleted the template-request-include-all-branches branch June 7, 2021 21:47
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
cla: yes Indication that the PR author has signed a Google Contributor License Agreement.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

2 participants