eclipse-ee4j / eclipse-ee4j/parsson

Ability to configure indentation

Open
#22 5 comments 2 reactions 0 assignees View on GitHub
Dominant language
Java
Stars
17
Forks
25
PR merge metrics
No merged PRs in 30d

Description

Indent is currently hardcoded:
https://github.com/eclipse-ee4j/jsonp/blob/9892c0658f9d55b819d39039a364df37d1d60202/impl/src/main/java/org/glassfish/json/JsonPrettyGeneratorImpl.java#L31

I prefer two spaces as indendation. To change I would have to replace the entire implementation. It should be trivial to create a configuration property so the indent could be set via the config map supplied to the generator factory.
WDYT?

Contributor guide

Open the contributing guide

Research direction

Start with impl/src/main/java/org/glassfish/json/JsonPrettyGeneratorImpl.java at the hardcoded indentation, then trace how the generator factory consumes its configuration map. Done means the indentation can be selected through that map, including two spaces, without replacing the implementation.

Written by the indexing model from the issue text.

Assessment

Tech stack
java
Domain
api
Issue type
Feature
Difficulty
3/5
Estimated time
1-2 days
Activity status
Stale
Clarity
Mostly clear
Newbie friendliness
45/100

Get new issues in your inbox

A short digest of beginner-friendly GitHub issues.