search

cps-org / cps

Common Package Specification — A cross-tool mechanism for locating software dependencies
https://cps-org.github.io/cps/
Other
99 stars 8 forks source link

Make attributes case-sensitive; change case #37

Closed mwoehlke closed 7 months ago

mwoehlke commented 7 months ago

In line with community feedback (see #2), make JSON keys case sensitive, and use all lower-case with underscores rather than title-case with hyphens. This mostly affects attribute names, but also some attributes that use language names as keys for language-specific values.

Needless to say, this is a major compatibility break.

Closes #2.

mwoehlke commented 7 months ago

Some probably overlooked details I noticed

Yup, I was noticing those also; thanks! (I found some others that didn't make your list, that I'll be fixing as well...)

mwoehlke commented 7 months ago

Oops, forgot sample.cps. (Hopefully I found everything else, though!)

mwoehlke commented 7 months ago

Thanks, @bretbrownjr! Since the EE meeting is tomorrow, I'll give this until then in case there is other feedback and plan on merging it after that.

bretbrownjr commented 7 months ago

We didn't explicitly discuss this in the C++ Ecosystem Evolution meeting, but it has been two days.

I'm happy to help merge PRs like this, but I don't have permissions at the moment.