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

Changed single-use to single_use #988

Closed
wants to merge 1 commit into from
Closed

Changed single-use to single_use #988

wants to merge 1 commit into from

Conversation

Kaegun
Copy link

@Kaegun Kaegun commented Aug 13, 2017

Looks like there's a typo in the Constants for Usage.

@remi-stripe
Copy link
Contributor

@anelder-stripe assigning this one to you, looks like this one should be merged in though I'm surprised we didn't have test errors before this fix.

@remi-stripe
Copy link
Contributor

r? @brandur-stripe @fred-stripe
Bumping this one to make sure it gets handled soon

@remi-stripe
Copy link
Contributor

@Kaegun Sorry for the delays on this one. We ship a lot of fixes today to the library. Would you mind rebasing your branch so that we can get it other the finish line?

Can you also add a test so that it confirms the change does work as expected?

@remi-stripe
Copy link
Contributor

Closing this one as @ob-stripe pushed a new one in #1084

@remi-stripe remi-stripe closed this Jan 3, 2018
@Kaegun
Copy link
Author

Kaegun commented Jan 29, 2018

Thank remi, can I delete the branch?

@remi-stripe
Copy link
Contributor

@Kaegun you can as it's been fixed and merged already!

@Kaegun Kaegun deleted the patch-1 branch January 29, 2018 19:53
@Kaegun
Copy link
Author

Kaegun commented Jan 29, 2018

Done. Thanks for fixing it! :-)

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.

2 participants