identify icon indicating copy to clipboard operation
identify copied to clipboard

Published 20 hours ago • verified publisher icon pre-commit

Fix: #334. Add .yamlld and .jsonld extensions.

Open ioggstream opened this issue 2 years ago • 3 comments

This PR

  • adds .yamlld and .jsonld extensions

ioggstream avatar Oct 04 '22 15:10 ioggstream

these should probably be classified as both their ld equivalents and the base config language as well in case one would want to lint those specifically

asottile avatar Oct 04 '22 16:10 asottile

Hi @asottile! I landed here from pre-commit.

  1. The desiderata is that jsonld and yamlld are at least linted with the JSON/yaml linters. It seems that check-yaml skips .yamlld files

  2. It is possible to add further validation (eh using jsonld) JSON schema but it is a complex task to be achieved consistently.

How do you think it is possible to achieve point 1?

Have a nice day!

ioggstream avatar Oct 05 '22 06:10 ioggstream

your patch does part 1 but you need to address my comment to get merged

asottile avatar Oct 05 '22 12:10 asottile

PTAL I "copied" from the xhtml entry. Have a nice day, R.

ioggstream avatar Oct 23 '22 19:10 ioggstream