search

cnabio / cnab-spec

Cloud Native Application Bundle Specification
https://cnab.io
Other
956 stars 99 forks source link

Remove outdated reference to `bundle.cnab` #353

Closed radu-matei closed 4 years ago

radu-matei commented 4 years ago

The 100 section apparently still had two references to a signed bundle being bundle.cnab.

This PR just removes those two references, but I suspect we could also add a note in the History section describing the changes.

The main question is whether this can be considered an errata change (see #351).

This PR is opened against the 1.0.1 branch - is this the right approach? How should master be updated?

cc @chris-crone @technosophos

vdice commented 4 years ago

This PR is opened against the 1.0.1 branch - is this the right approach? How should master be updated?

Perhaps we either cherry-pick from the 1.0.1 branch... or start via a PR against master and then cherry-pick into the 1.0.1 branch (reverse)?

carolynvs commented 4 years ago

+1 for this being treated as a patch since this wasn't implementable

technosophos commented 4 years ago

@vdice are you okay with me merging this onto master?

vdice commented 4 years ago

@technosophos 👍