Skip to content

fix(jsonify): parse returning undefined instead of the number#2738

Closed
DevJoaoLopes wants to merge 8 commits intodenoland:mainfrom
DevJoaoLopes:fix/parse-objetc-with-negative
Closed

fix(jsonify): parse returning undefined instead of the number#2738
DevJoaoLopes wants to merge 8 commits intodenoland:mainfrom
DevJoaoLopes:fix/parse-objetc-with-negative

Conversation

@DevJoaoLopes
Copy link
Contributor

close #2736

Copy link
Collaborator

@marvinhagemeister marvinhagemeister left a comment

Choose a reason for hiding this comment

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

Great catch, thanks for making a PR.

@DevJoaoLopes DevJoaoLopes force-pushed the fix/parse-objetc-with-negative branch from d0f96f4 to f3cc496 Compare October 30, 2024 12:51
@DevJoaoLopes
Copy link
Contributor Author

Taking advantage, I would like to know if it would be possible to join the list of contributors to the project. I've been using deno and fresh in my daily life and I've been trying more and more to contribute here. What do you think @marvinhagemeister ?

@DevJoaoLopes
Copy link
Contributor Author

@marvinhagemeister Can we merge this PR?

@DevJoaoLopes
Copy link
Contributor Author

@marvinhagemeister @lucacasonato How can we proceed with this PR 😄 ?

@DevJoaoLopes
Copy link
Contributor Author

@marvinhagemeister @lucacasonato Can you take a look at this MR if it still makes sense please?

@marvinhagemeister
Copy link
Collaborator

Apologies, looks like this was addressed in #2790

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.

jsonify parse -1 into undefined

2 participants