Skip to content

Add a more maintaned Python, and new Java and Kotlin JSON libraries t… - #11

Merged
DecimalTurn merged 3 commits into
JSONC-org:mainfrom
vorburger:patch-1
Oct 7, 2025
Merged

DecimalTurn merged 3 commits into
JSONC-org:mainfrom
vorburger:patch-1

Conversation

@vorburger

Copy link
Copy Markdown
Contributor

…o index

@DecimalTurn

DecimalTurn commented Oct 5, 2025 •

Copy link
Copy Markdown
Member

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 https://github.com/DecimalTurn/JSONC/issues/10#issuecomment-3346673830

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

@DecimalTurn

Copy link
Copy Markdown
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
Copy Markdown
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.

@DecimalTurn DecimalTurn left a comment

Copy link
Copy Markdown
Member

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.

Comment thread index.markdown Outdated
@DecimalTurn
DecimalTurn merged commit 8a0b735 into JSONC-org:main Oct 7, 2025
1 check failed
@DecimalTurn

Copy link
Copy Markdown
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