Page MenuHomeDevCentral

Add command to validate configuration files
ClosedPublic

Authored by dereckson on Apr 9 2017, 14:36.
Tags
None
Referenced Files
F3753217: D949.diff
Mon, Nov 18, 00:01
Unknown Object (File)
Tue, Nov 12, 11:26
Unknown Object (File)
Thu, Nov 7, 10:23
Unknown Object (File)
Fri, Oct 25, 17:51
Unknown Object (File)
Oct 19 2024, 13:31
Unknown Object (File)
Oct 17 2024, 02:54
Unknown Object (File)
Oct 9 2024, 15:02
Unknown Object (File)
Sep 22 2024, 18:51
Subscribers
None

Details

Summary

Currently, the command validates JSON syntax,
using the own PHP parser instead to rely on jsonlint,
to avoid to depends of Node.JS for one command.

Fixes T1177.

Test Plan

New tests check command output is expected when:

  • all the files are valid
  • there is a general syntax error
  • there is a specific syntax error

Diff Detail

Repository
rNOTIF Notifications center
Lint
Lint Passed
Unit
Tests Passed
Branch
validate-json
Build Status
Buildable 1469
Build 1717: arc lint + arc unit

Event Timeline

dereckson edited the test plan for this revision. (Show Details)
dereckson edited the summary of this revision. (Show Details)
This revision is now accepted and ready to land.Apr 9 2017, 14:38
dereckson edited the test plan for this revision. (Show Details)

Style

This revision was automatically updated to reflect the committed changes.