Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

close JsonWriters #148

Closed
wants to merge 1 commit into from
Closed

close JsonWriters #148

wants to merge 1 commit into from

Conversation

gjvanderheiden
Copy link
Contributor

Pull Request Checklist

Please confirm that you've done the following when opening a new pull request:

  • For fixes and other improvements, please reference the GitHub issue that your change addresses.
  • For fixes, optimizations and new features, please add an entry to the CHANGES.md file.
  • Run mvn compile before committing, so that the auto-code formatter will format your changes consistently with the rest of the project.

Closing jsonwriters. My local build reacts faster and less "device not responding" issues. Updated a bunch of dependencies and used my own pr about updating json blabla. But closing surely doesn't hurt.

@gjvanderheiden
Copy link
Contributor Author

Seen / stole this in/from #116. Searched on createWriter and found 3 more occurrences.

@gjvanderheiden gjvanderheiden deleted the close_json_writer branch January 2, 2022 15:23
@yfre
Copy link
Contributor

yfre commented Jan 27, 2022

@gjvanderheiden can we reopen this PR? i would like to merge it

@yfre yfre mentioned this pull request Feb 5, 2022
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