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

Use go.mercari.io/go-emv-code import path #1

Merged
merged 10 commits into from
Jun 30, 2019
Merged

Use go.mercari.io/go-emv-code import path #1

merged 10 commits into from
Jun 30, 2019

Conversation

zchee
Copy link
Contributor

@zchee zchee commented Jun 28, 2019

What

Use go.mercari.io/go-emv-code import path.

Why

We have already deployed the go-emv-code package to go.mercari.io/go-emv-code.
Use the go.mercari.io/go-emv-code/... custom import path instead of.

@zchee zchee added the enhancement New feature or request label Jun 28, 2019
@zchee zchee requested a review from ymt2 June 28, 2019 09:20
@zchee zchee self-assigned this Jun 28, 2019
@zchee
Copy link
Contributor Author

zchee commented Jun 28, 2019

@ymt2 Should I squash to one commit? up to you.

@ymt2
Copy link
Contributor

ymt2 commented Jun 29, 2019

No, u don't need to 😄

doc.go Outdated Show resolved Hide resolved
@zchee
Copy link
Contributor Author

zchee commented Jun 29, 2019

@ymt2 PTAL.

Copy link
Contributor

@ymt2 ymt2 left a comment

Choose a reason for hiding this comment

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

LGTM! Thx!

@ymt2 ymt2 merged commit 6721926 into master Jun 30, 2019
@ymt2 ymt2 deleted the use-go-mercari-io branch June 30, 2019 09:31
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
enhancement New feature or request
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

2 participants