• Home
  • Features
  • Pricing
  • Docs
  • Announcements
  • Sign In

mozilla / addons-linter / 5044
100%

Build:
DEFAULT BRANCH: master
Ran 06 Feb 2018 02:13PM CUT
Jobs 1
Files 61
Run time 3s
Badge
Embed ▾
README BADGES
x

If you need to use a raster PNG badge, change the '.svg' to '.png' in the link

Markdown

Textile

RDoc

HTML

Rst

pending completion
5044

push

travis-ci

EnTeQuAk
Rules for messages.json (#1801)

Fixes #1666

This adds a parser for messages.json files. The parser is called from the JSON scanner. It has a JSON schema for the messages.json files, which is currently just in the schema's directory, since it is neither imported nor a patch of an existing schema. I'm pretty sure that's not where it should be, though.

It also warns about missing placeholders and when using reserved message names (which are potentially inaccessible).

I think the naming regexp as it is documented is not enforced like this by Firefox, so it may be demoted to a warning, since the extension will still work, even with characters other than [A-Za-Z0-9_@] in message names. I don't know about placeholders, not something I have violated myself.

The errors are kind of annoying currently, since you have no position or reference to which message they happen at. I haven't really found precedent for how to handle that though.

And lastly, the documentation links should probably be shortened, so they're actually clickable.

* Rules for messages.json
* Add lowercase handling and add locale perfix for messages.json
* Add test for getLowercasePlaceholders when there are no placeholders

948 of 986 branches covered (96.15%)

1671 of 1671 relevant lines covered (100.0%)

430.83 hits per line

Jobs
ID Job ID Ran Files Coverage
1 5044.1 06 Feb 2018 02:13PM CUT 0
100.0
Travis Job 5044.1
Source Files on build 5044
  • List 0
  • Changed 27
  • Source Changed 5
  • Coverage Changed 27
Loading...
Coverage∆FileLinesRelevantCoveredMissedHits/LineBranch HitsBranch Misses
No data available in table
Showing 0 to 0 of 0 entries
  • Previous
  • Next
  • Back to Repo
  • Travis Build #5044
  • 3923b399 on github
  • Prev Build on master (#5043)
  • Next Build on master (#5045)
STATUS · Troubleshooting · Open an Issue · Sales · Support · CAREERS · ENTERPRISE · START FREE · SCHEDULE DEMO
ANNOUNCEMENTS · TWITTER · TOS & SLA · Supported CI Services · What's a CI service? · Automated Testing

© 2025 Coveralls, Inc