Seldaek / jsonlint

JSON Lint for PHP

Geek Repo:Geek Repo

Github PK Tool:Github PK Tool

Does it support PHP 7.1 and 7.2 too?

andkirby opened this issue · comments

Hi!
I guess your code does support. :)
I just see some limitations in composer.json file.
Could you test your code and update requirements?

Thanks for the module, by the way!

Off-topic...
@Seldaek , you may simplify path src/Seld/JsonLint. Just update composer.json to PSR-4 for \Seld\JsonLint\ namespace.

  "autoload": {
    "psr-4": {
      "\\Seld\\JsonLint\\\": "src/"
    }
  },

...and put files from src/Seld/JsonLint into src/.

I don't see what you mean by problems in composer.json - but yes it does support 7.1/7.2. The library is used in Composer itself which runs on all new php versions for sure. As for PSR-4 I am well aware but I don't see the need to change this as it doesn't really need changing.

Yep, I just missed "^" sign near "7.0" in versions requirement.
Thx for the answer.