Skip to content

Conversation

@igorpeshansky
Copy link
Contributor

No description provided.

@igorpeshansky igorpeshansky marked this pull request as ready for review September 2, 2021 20:54
Copy link
Contributor

@qingling128 qingling128 left a comment

Choose a reason for hiding this comment

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

LGTM

Copy link
Contributor Author

@igorpeshansky igorpeshansky left a comment

Choose a reason for hiding this comment

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

Thanks, PTAL.

@qingling128
Copy link
Contributor

That answered all my questions. Feel free to submit.

Copy link
Contributor

@qingling128 qingling128 left a comment

Choose a reason for hiding this comment

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

The new name LGTM

@igorpeshansky igorpeshansky merged commit a09618e into master Sep 3, 2021
@igorpeshansky igorpeshansky deleted the igorpeshansky-github-actions branch September 3, 2021 03:22
go install github.com/mattn/goveralls@latest
- run: |
git config --global core.symlinks true
- uses: actions/checkout@v2
Copy link
Contributor Author

Choose a reason for hiding this comment

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

igorpeshansky added a commit that referenced this pull request Sep 8, 2021
Travis used to do that by default. This fixes a migration regression from #189.
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.

3 participants