search

martinthomson / i-d-template

A template for IETF internet draft git repositories
Other
212 stars 183 forks source link

add CDDL validation and CBOR diagnostic tools #250

Closed thomas-fossati closed 4 years ago

thomas-fossati commented 4 years ago

This adds cddl(1) and cbor-diag(1) to the CI docker image toolchain.

It allows validating inlined CDDL and checking CBOR examples against CDDL.

An example usage is the RATS EAT spec:

It looks to me like it could be generally useful?

cheers!

thomas-fossati commented 4 years ago

However, I've been working on a process for tools like this - which aren't used by every document - to be integrated into the process. We could gather a list of tools like this and put them all in these images, but that would bloat the images quite a bit. Instead, I've put together a guide on how tools can be integrated on an as-needed basis. I think that this is better.

Thanks, I hadn't seen this, my bad. Looks pretty neat.

Over time, it's possible that widespread use of particular tools means that it will be easier to have them integrated into these images, but I would prefer to have people develop experience with adding additional validation tools.

Agree. The image that includes these is 319M -- vs 310M of the original, a ~3% increase for just a couple of tools that not everyone is going to use is clearly sub-optimal.