Skip to content
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

tag v1.1.0-rc.3 #1049

Merged
merged 2 commits into from
Apr 27, 2023
Merged

tag v1.1.0-rc.3 #1049

merged 2 commits into from
Apr 27, 2023

Conversation

vbatts
Copy link
Member

@vbatts vbatts commented Apr 19, 2023

to tag

this is would mean tagging 085b884 as v1.1.0-rc.3 (now with more SemVer)

changes

fd45b6b Add scratch descriptor and scope layer limits
63b8bd0 Remove artifact manifest
23c4647 Define image manifest artifactType and guidance
5751791 Add Tianon as maintainer
f4fc83a Fix unused variable linting error
d09d13d Update Jon Johnson's email
4136bec descriptor schema: add missing data and artifactType definitions
729a03e manifest, specs-go/: provide guidance on SCRATCH config descriptor
31de013 manifest schema: add tests for the subject field
7a9efbd manifest schema: add the missing subject field
f2f1956 descriptor: clarify artifactType field must have compliant values
98f35df Update image spec and conversion to clarify groups
336b02c Require IANA mediaType for image config.mediaType and layers.mediaType
1f60184 Add Go 1.20 support
f99b121 Remove filtersApplied from image-spec
b5998ba specs-go/v1/*.go: align the deprecation style
6687119 Chore: fix go.mod - split direct/indirect dependencies
867ce74 ArtifactType is optional, omit when empty
ccb86b9 mention deprecation in media-types.md
9b4e6c0 even fewer words
2cdbef2 Deprecate non-distributable layers
265874e Note an exception to the platform.os recommendation for wasi
59780aa Add ArgsEscaped field to image config
3625ee3 doc: fix example in artifact.md
94f2431 version: bump main back to -dev
0a97fe7 docs: Added artifact.md to docs and spec.md
ca2e500 Embed Platform in Image
293f064 Reverting json schema to well known value

cyphar

This comment was marked as outdated.

@@ -25,7 +25,7 @@ const (
VersionPatch = 0

// VersionDev indicates development branch. Releases will be empty string.
VersionDev = "-dev"
VersionDev = "-rc.3-dev"
Copy link
Member

@cyphar cyphar Apr 21, 2023

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Actually, I just remembered -- using -dev this way results in suboptimal behaviour with SemVer. SemVer treats 1.0.0-dev as being a pre-release of 1.0.0 -- meaning that the -dev version is seen as "older". Another issue is that -dev is not really a pre-release (and it's definitely not a pre-release of the previously-released version).

In runc and umoci we use +dev because that matches the correct version order behaviour with rpm and most other tools (1.0.1 > 1.0.0+dev > 1.0.0), though under SemVer 1.0.0+dev == 1.0.0 (I don't think this is a problem for a specification though, but maybe that's just me).

Maybe we should use +dev as well?

Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Ah, this is outlined in RELEASES.md. I will submit a PR to make the change for future releases.

Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

See #1050.

@jonjohnsonjr jonjohnsonjr merged commit 77d92ec into opencontainers:main Apr 27, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

7 participants