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

Fixed syntax error in #53 #60

Closed
wants to merge 2 commits into from

Conversation

joeyfigaro
Copy link

#53

@BeckyWu220
Copy link
Contributor

Hi @joeyfigaro Thanks for contributing to the library. Could you please update the PR by removing the package.lock file and the .editorconfig file? Thanks!

@BeckyWu220 BeckyWu220 force-pushed the master branch 4 times, most recently from 42c67ac to 8693ae4 Compare March 27, 2020 01:02
@BeckyWu220
Copy link
Contributor

Hi! @joeyfigaro Sorry for responding late to your PR. As you might had noticed, we had some new PRs merged. One of the PR had provided a typescript implementation, and this issue had been fixed in the same PR. You should not encounter this issue again in the next release (v1.3.x). Please feel free to test again in the new version that would be released soon.

@BeckyWu220 BeckyWu220 force-pushed the master branch 4 times, most recently from 89f120f to 29494b9 Compare March 27, 2020 21:09
@joeyfigaro
Copy link
Author

@BeckyWu220 no worries! is this good to close, then?

@BeckyWu220
Copy link
Contributor

Hi @joeyfigaro Yes. I will go ahead and close the PR. Thanks for your contribution.

@BeckyWu220 BeckyWu220 closed this Apr 2, 2020
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