[wp-trac] [WordPress Trac] #64221: Reclassify `json2php` as a `devDependency`

[WordPress Trac]

#64221: Reclassify `json2php` as a `devDependency`
--------------------------------------+------------------------
 Reporter:  desrosj                   |       Owner:  desrosj
     Type:  task (blessed)            |      Status:  reviewing
 Priority:  normal                    |   Milestone:  7.0
Component:  Build/Test Tools          |     Version:
 Severity:  normal                    |  Resolution:
 Keywords:  good-first-bug has-patch  |     Focuses:  tests
--------------------------------------+------------------------
Changes (by dilip2615):

 * focuses:   => tests


Comment:

 Hello @desrosj, I reviewed the patch in PR #10520, and it looks correct.
 `json2php` is only used in the build tooling (`copy:block-json`) and not
 needed at runtime, so moving it from `dependencies` to `devDependencies`
 makes sense.

  **Tested locally:**

 * `npm install` (lockfile updated as expected)
 * `npm run build` (no errors)
 * `npx grunt copy:block-json` (task runs successfully).
  => +1 to commit/merge. Any PHPUnit deprecation warnings I saw appear
 unrelated to this change.

-- 
Ticket URL: <https://core.trac.wordpress.org/ticket/64221#comment:7>
WordPress Trac <https://core.trac.wordpress.org/>
WordPress publishing platform