Skip to content

Make semver checks exclude unstable types - #204

Merged
wfraser merged 5 commits into
dropbox:masterfrom
wfraser:semver-checks-exclude-unstable-types
Jul 27, 2026
Merged

Make semver checks exclude unstable types#204
wfraser merged 5 commits into
dropbox:masterfrom
wfraser:semver-checks-exclude-unstable-types

Conversation

@wfraser

@wfraser wfraser commented Jul 25, 2026

Copy link
Copy Markdown
Member

This partially fixes #111, specifically the "don't test unstable things" part. I haven't extended it to deprecated things yet.

Checklist

General Contributing

  • I have read the Code of Conduct and signed the CLA.
  • I have added an entry to the RELEASE_NOTES.md file, or believe it's not necessary for this change.

Validation

The current pending API spec update makes a lot of breaking changes, but they're all in the riviera namespace, which is all unstable. The current update_spec.py fails and says it's semver-incompatible, but with this change, the tests of the riviera namespace types are config'd out, and the tests pass.

this is only really needed on Windows, where Python likes to assume we're using CP-1252 like it's 1997 or something
@wfraser wfraser changed the title Semver checks exclude unstable types Make semver checks exclude unstable types Jul 25, 2026
@wfraser
wfraser force-pushed the semver-checks-exclude-unstable-types branch from e9fc675 to 3994377 Compare July 26, 2026 03:06
@wfraser
wfraser force-pushed the semver-checks-exclude-unstable-types branch from 3994377 to 76ffa63 Compare July 27, 2026 19:09
@wfraser
wfraser merged commit 6d9d99c into dropbox:master Jul 27, 2026
3 checks passed
@wfraser
wfraser deleted the semver-checks-exclude-unstable-types branch July 27, 2026 19:13
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.

on spec update, breakage in types used only by preview routes (and types unused by any routes) should be ignored for semver purposes

1 participant