Azeem Bande-Ali
Azeem Bande-Ali
Cool, thanks for the explanation! In my opinion, this is not as much about correctness as it is about tradeoffs. The problem is that we don't want eslint to apply...
Thanks for the link to the RFC! It is particularly helpful since `eslint-plugin-json` is explicitly mentioned as a plugin that can benefit from this design. @ota-meshi If we wanted `eslint-plugin-json`...
Since the [spec](https://www.rfc-editor.org/rfc/rfc7159#section-8.1) says that the BOM shouldn't be added, I think its fair for a linting tool to complain about it. > Implementations MUST NOT add a byte order...
Thanks a lot for opening this PR! I honestly haven't kept up with eslint so I will first try to familiarize myself with what eslint is changing and then test...
I made changes to the test setup on the main branch so rebased here and force pushed.
@BePo65 Am I understand correctly that the current version of eslint-plugin-json fails when running with eslint v9? If so, thanks for flagging that! As @Standard8 suggested, I think we can...
I will merge this PR now but won't push a release immediately. I will try to create a patch to add the schema support based on the snippet provided @BePo65....
This work has been released as [v4.0.0](https://www.npmjs.com/package/eslint-plugin-json/v/4.0.0)!
Sorry for the delay here. Here is a demo video: https://user-images.githubusercontent.com/2160795/206884873-a197224b-106e-44cb-a1d7-ab1e41b49873.mp4 Here is what the content of the file looks like (after running a JSON prettifier on the data) ```json...
I understand the hesitance with forcing the wrapping on the user but I would appreciate a keyboard shortcut or a button to wrap the current message. That way folks that...