-
Notifications
You must be signed in to change notification settings - Fork 645
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Serialization.md needs a big overhaul #36
Comments
@vbatts I am ready to merge this improved doc after reviewing it. Are we blocked on that PR or can we merge it as-is and then add fixes as they go? |
hmm. i'm okay with merging and adding fixes as we/they go |
@vbatts you got dis or do you want me to do it? Lets wait a day or so to merge it. |
k. I'll open a PR |
@vbatts this stuff replaces the config in serialization.md? |
not so much replaces, but updates |
@vbatts are we waiting on anything or can this be closed out? |
Moving to v0.2.0 as I don't know what we are blocked on or what updates we need. Help would be appreciated @vbatts. |
@vbatts moby/moby#22264 has been merged. So this should be easy?! |
The diff is very trivial from moby/moby#22264 to the OCI version and has no material changes. I am going to close this issue out and open a PR for the one wording improvement. Word diff: https://gist.github.com/philips/684f2514b071fff1b6d4b68a36747432 |
Closing PR: #91 |
Per #26 (comment) and
https://github.com/opencontainers/image-spec/issues?q=is%3Aissue+is%3Aopen+label%3A%22component%2Fserialization+spec%22
There is much cruft carried forward with the version that was cherry-picked for this image-spec.
The text was updated successfully, but these errors were encountered: