Skip to content

Add single_line_array param to print array in single line#124

Closed
hongquan wants to merge 1 commit into
python-rapidjson:masterfrom
hongquan:feature/single-line-array
Closed

Add single_line_array param to print array in single line#124
hongquan wants to merge 1 commit into
python-rapidjson:masterfrom
hongquan:feature/single-line-array

Conversation

@hongquan

Copy link
Copy Markdown

Try solving #123

@lelit

lelit commented Oct 29, 2019

Copy link
Copy Markdown
Contributor

Looks good! I won't be able to test it out right now, but will do as soon as possible.
Maybe someone can investigate on that quite terse failure on AppVeyor?

lelit added a commit that referenced this pull request Nov 2, 2019
This satisfies issue #123 in a different way than the one proposed in PR #124, slightly
simpler and open to other possible formatting options.
@lelit

lelit commented Nov 2, 2019

Copy link
Copy Markdown
Contributor

Hi, I opted for a slightly different API to implement the feature, a tad simpler and open to possible other PrettyWriter variants.
Thank you for the idea and initial implementation!

@lelit lelit closed this Nov 2, 2019
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