Add ignoreFontFamilies option to font-family-no-missing-generic-family-keyword

[mattfelten]

Which issue, if any, is this issue related to?

Closes #3288

Is there anything in the PR that needs further explanation?

Seems to work alright so far. Something I'm struggling with is how a font family name in quotes should be written. I couldn't tell if that works and I'm just writing the test poorly or this doesn't work with quoted names.

[jeddy3]

@mattfelten Thank you for your pull request!

I now realise that the option should be called ignoreFontFamilies as value is ambiguous and does not take into account font families within the value of the font shorthand property. Can you update the pull request to reflect this, please?

I've also requested a couple of other changes to bring the option in line with those in other rules.

[mattfelten]

@jeddy3 Thanks for the review. Just made the requested changes. Let me know if I can make this better any more.

[jeddy3]

@mattfelten Thanks for making the amends. It's looking good!

I've made two more, very minor, requests.

[mattfelten]

@jeddy3 Thanks! Got those taken care of too

[jeddy3]

@mattfelten Thanks for making the changes so promptly. For some reason, I didn't get a notification so I'm only just looking now.

We'll make sure this gets into our next pending release.

Thanks again!

[ybiquitous]

I found some typos and suggest one on the rule document.

[ybiquitous]

[suggestion] Is it reasonable to add test cases for regex?

[mattfelten]

Thanks @jeddy3 !

@ybiquitous I got your changes in. Thanks for looking! Lmk if I can improve anything else.

[ybiquitous]

LGTM!👍

[jeddy3]

Changelog:

• Added: ignoreFontFamilies: [] to font-family-no-missing-generic-family-keyword (#4656).