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

Relax semver-checks to compare worlds in packages with different versions #1886

Conversation

sunfishcode
Copy link
Member

In wasm-tools component semver-checks, instead of requiring that the old and new world live in packages with the same id, just require that they have the same name. This allows checking for semver compatibility between different versions of a package, for example, this command:

$ wasm-tools component semver-check --prev wasi:http/[email protected] --new wasi:http/[email protected] ./wit

in a dirctory with wit/deps/[email protected] contains the current wasi-http wit files, and wit/deps/[email protected] contains a new set of wit files, checks that the new version is semver-compatible.

…ions

In wasm-tools component semver-checks, instead of requiring that the old
and new world live in packages with the same id, just require that they
have the same name. This allows checking for semver compatibility
between different versions of a package, for example, this command:

```shell
$ wasm-tools component semver-check --prev wasi:http/[email protected] --new wasi:http/[email protected] ./wit
```

in a dirctory with `wit/deps/[email protected]` contains the current wasi-http
wit files, and `wit/deps/[email protected]` contains a new set of wit files,
checks that the new version is semver-compatible.
@alexcrichton alexcrichton added this pull request to the merge queue Oct 30, 2024
Merged via the queue into bytecodealliance:main with commit adbc32f Oct 30, 2024
30 checks passed
@sunfishcode sunfishcode deleted the sunfishcode/semver-check-different-versions branch October 30, 2024 20:41
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.

2 participants