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.

Sign up for GitHub

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

Jump to bottom

Merge develop into master #141

Merged
merged 9 commits into from
Nov 30, 2020
Merged

Merge develop into master #141

merged 9 commits into from
Nov 30, 2020

Conversation

cweill
Copy link
Owner

@cweill cweill commented Nov 25, 2020 •
edited

Fixes #135

ZaynJarvis and others added 8 commits November 13, 2019 22:37
@ZaynJarvis @cweill
Support to go modules (#113)
1c5384d
@shihanng @cweill
Run go mod tidy (#117)
62fe1b8
@178inaba
Fix gofmt warning (#129)
205cc46 
Run `gofmt -s -w .`.

Fixes #128
@178inaba @cweill
Add 1.13.x to .travis.yml (#127)
0437483 
Fixes #126

Co-authored-by: Charles Weill <c@weill.io>
@178inaba @cweill
Fix misspell warning (#123)
b462be8 
Fixes #122

Co-authored-by: Charles Weill <c@weill.io>
@178inaba @cweill
Fix vet warning (#125)
cf73e22 
Fixes #124

Co-authored-by: Charles Weill <c@weill.io>
@shihanng @cweill
Add GOTESTS_TEMPLATE and GOTESTS_TEMPLATE_DIR environment variables (#…
7cb3848 
…118)

* Read from environment variables

For the following flags

- template

- template_dir

* Add docs about setting environment variables

* Fix envvar-flag precedence

* Add proper docs on flags

* Add test cases for valOrGetenv

Co-authored-by: Charles Weill <c@weill.io>
@178inaba
Add parallel to sub test (#121)
2435ae5 
* Add parallel to sub test

* Add parallel flag and option

* Fix README

* Add parallel to template

* Fix templates

* Add tests
@googlebot
Copy link
Collaborator

All (the pull request submitter and all commit authors) CLAs are signed, but one or more commits were authored or co-authored by someone other than the pull request submitter.

We need to confirm that all authors are ok with their commits being contributed to this project. Please have them confirm that by leaving a comment that contains only @googlebot I consent. in this pull request.

Note to project maintainer: There may be cases where the author cannot leave a comment, or the comment is not properly detected as consent. In those cases, you can manually confirm consent of the commit author(s), and set the cla label to yes (if enabled on your project).

ℹ️ Googlers: Go here for more info.

@cweill cweill self-assigned this Nov 25, 2020
@cweill cweill added cla: yes Contributor signed Google CLA and removed cla: no labels Nov 25, 2020
@googlebot
Copy link
Collaborator

A Googler has manually verified that the CLAs look good.

(Googler, please make sure the reason for overriding the CLA status is clearly documented in these comments.)

ℹ️ Googlers: Go here for more info.

@coveralls
Copy link

Coverage Status

Coverage increased (+0.02%) to 96.244% when pulling dcd80c9 on develop into 43a47fb on master.

@cweill cweill merged commit d36138c into master Nov 30, 2020
@cweill cweill mentioned this pull request Nov 30, 2020
Closed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers
No reviews
Assignees

@cweill cweill

Labels
cla: yes Contributor signed Google CLA
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.

None yet
6 participants
@cweill @googlebot @coveralls @ZaynJarvis @shihanng @178inaba