Add 'json' to the list of extensions for 'js'

[rwstauner]

I was grepping in the directory of an extracted CPAN distribution
for module dependencies and noticed that META.yml had a hit while META.json didn't show up.

I almost made json it's own type, then thought maybe it should just be in with the 'js'...
Any thoughts on the matter?
Or perhaps have you specifically left json out for a reason?

[crccheck]

My first thought is that json would be its own. Five examples:

• JSON files usually don't have line breaks
• yaml is its own type
• say you have a JSON that's some sort of massive dictionary. Imagine searching for a variable name in your JS file.

(there's a Monty Python joke in here somewhere)

[rwstauner]

yeah I thought of those too...
guess I should have trusted my first instinct.

I see now that ack and ack-help-types.txt have been modified (didn't notice it last time... maybe I didn't "make" it right).

[llimllib]

+1, either as its own type, or as part of js

[rwstauner]

Note that in the meantime you can put

--type-add=json=.json

in your ~/.ackrc to include .json files in searches

[rwstauner]

or

--type-add=js=.json

to avoid the warning about making a new type

[dolmen]

@rwstauner To define a new type, use --type-set instead of --type-add:

--type-set json=.json

https://metacpan.org/module/ack#Defining-your-own-types

[hoelzro]

This made it into ack2, so closing!