Skip to content

Conversation

@vorburger
Copy link
Contributor

…o index

@DecimalTurn
Copy link
Member

DecimalTurn commented Oct 5, 2025

We are currently in the process of adding more information about packages regarding trailing commas support. Can you please confirm whether or not the packages you are suggesting to add support trailing commas?

See #10 (comment)

PS: Don't worry about the CI preview failing, it's not required for merging.

@DecimalTurn
Copy link
Member

Looking at NickolaiBeloguzov/jsonc-parser#3 makes me think we might want to drop the other Python package completely actually.

@DecimalTurn DecimalTurn changed the title Add a more maintaned Python, and nwe Java and Kotlin JSON libraries t… Add a more maintaned Python, and new Java and Kotlin JSON libraries t… Oct 5, 2025
@vorburger
Copy link
Contributor Author

Can you please confirm whether or not the packages you are suggesting to add support trailing commas?

Looking at NickolaiBeloguzov/jsonc-parser#3 makes me think we might want to drop the other Python package completely actually.

SGTM, will update this PR with that now.

Copy link
Member

@DecimalTurn DecimalTurn left a comment

Choose a reason for hiding this comment

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

Looks good, just need to address the name of the Kotlin library.

@DecimalTurn DecimalTurn merged commit 8a0b735 into JSONC-org:main Oct 7, 2025
1 check failed
@DecimalTurn
Copy link
Member

Thanks!

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