ethpm / ethpm-spec

Ethereum Package Manager http://ethpm.github.io/ethpm-spec/
165 stars 30 forks source link

Enforce top-level `manifest_version` field is illegal #153

Closed njgheorghita closed 4 years ago

njgheorghita commented 4 years ago

V2 defined the manifest version in a "manifest_version" field. V3 defines the manifest version in a "manifest" field. In order to increase the delineation between V2 & V3 manifests, it would be useful to make it illegal to use a top-level key of "manifest_version" in V3 manifests. While it's unlikely that users would add a user-defined field of "manifest_version" to a V3 manifest, explicitly forbidding this field will make it easier for machines to distinguish b/w V2 & V3 manifests.