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

Handle corner case of values that look like numbers but should be treated as strings #51

Merged
merged 3 commits into from Dec 4, 2023

Conversation

rogerluan
Copy link
Owner

Resolves #50

@rogerluan rogerluan added the bug label Nov 16, 2023
@rogerluan rogerluan self-assigned this Nov 16, 2023
Copy link

@dcacenabes dcacenabes left a comment

Choose a reason for hiding this comment

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

Amazing! We are also struggling with this issue, looking forward to seeing it fixed :)

@rogerluan
Copy link
Owner Author

Merging this as I didn't hear back from the issue author for a few weeks 🙇 we can always go back and adjust this as needed :)

Thanks for the feedback @dcacenabes 🙌

@rogerluan rogerluan merged commit 35f9cff into main Dec 4, 2023
10 checks passed
@rogerluan rogerluan deleted the roger/handle-number-looking-strings branch December 4, 2023 17:03
@dcacenabes
Copy link

No no no, don't thank me. Thank you for the tool and for providing a fix!! I will closely watch for the release to update our dependency. We were in 1.1.1 and I just noticed the issue when we tried to bump to the latest version :)

Looking forward to the new version getting released!

@rogerluan
Copy link
Owner Author

haha no problem! ❤️

Aaaand it's out! https://github.com/rogerluan/arkana/releases/tag/v1.6.0 😊

Lemme know if you end up facing any issues! 🙇

@dcacenabes
Copy link

Just updated it, worked like a charm, could finally build the app :D Thanks again!

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Number string is converted to int
2 participants