azeemba / eslint-plugin-json

Lint your JSON files
MIT License
209 stars 29 forks source link

npm readme gives wrong config format in Usage section #92

Closed adamlui closed 2 months ago

adamlui commented 4 months ago

It shows

{
    export default [
      {
        files: ["**/*.json"],
        ...json.configs["recommended"],
    ];
}

...but should be like the format in GitHub readme without missing closing curly brace

export default [
  {
    files: ["**/*.json"],
    ...json.configs["recommended"]
  }
];
fnogatz commented 2 months ago

Note that this has already been resolved (see current version of README.md) as part of https://github.com/azeemba/eslint-plugin-json/pull/89, it just has not yet been pushed to npm as well. Maybe you could do so, @azeemba, so we can close this issue here?

azeemba commented 2 months ago

Whoops, thanks. Will do so later this evening

adamlui commented 2 months ago

Note that this has already been resolved (see current version of README.md)

I know that GitHub README has the correct instruction (I said that already in my OP) the issue is on npm it doesn't, so the specific issue "npm readme gives wrong config" is not actually resolved

azeemba commented 2 months ago

Thanks for the report and the update everyone!

There was no way to update the readme without a new published version. So I have published 4.0.1 with the updated readme.