Skip to content

Change the default JSON schema version to v0. #530

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

Merged
merged 3 commits into from
Jul 10, 2024

Conversation

grynspan
Copy link
Contributor

@grynspan grynspan commented Jul 10, 2024

This PR changes the default JSON schema version for event stream output to version 0 (the version documented in #479.) Previously, if a caller did not specify a version number for the event stream, it would default to encoding "snapshots" of various values from the testing library.

Xcode 16 Beta 1 uses this legacy format, so to continue supporting newer betas of Xcode 16 transitionally, this PR temporarily enables a "-1" JSON schema version that continues to emit snapshot encodings. This JSON schema version will be removed (along with the snapshots code in general) in a future update once Xcode has migrated.

Resolves #527.

Checklist:

  • Code and documentation should follow the style of the Style Guide.
  • If public symbols are renamed or modified, DocC references should be updated.

@grynspan grynspan added enhancement New feature or request tools integration Integration of swift-testing into tools/IDEs labels Jul 10, 2024
@grynspan grynspan self-assigned this Jul 10, 2024
@grynspan
Copy link
Contributor Author

@swift-ci please test

grynspan added 2 commits July 10, 2024 09:02
This PR changes the default JSON schema version for event stream output to
version 0 (the version documented in #479.) Previously, if a caller did not
specify a version number for the event stream, it would default to encoding
"snapshots" of various values from the testing library.

Xcode 16 Beta 1 uses this legacy format, so to continue supporting newer betas
of Xcode 16 transitionally, this PR temporarily enables a "-1" JSON schema
version that continues to emit snapshot encodings. This JSON schema version will
be removed (along with the snapshots code in general) in a future update once
Xcode has migrated.

Resolves #527.
@grynspan grynspan force-pushed the jgrynspan/527-default-json-schema branch from 766d4f9 to b376f4f Compare July 10, 2024 13:04
@grynspan
Copy link
Contributor Author

@swift-ci please test

@grynspan
Copy link
Contributor Author

@swift-ci please test Linux

@grynspan grynspan requested a review from stmontgomery July 10, 2024 13:36
@grynspan
Copy link
Contributor Author

@swift-ci please test

@grynspan grynspan merged commit 22d7705 into main Jul 10, 2024
3 checks passed
@grynspan grynspan deleted the jgrynspan/527-default-json-schema branch July 10, 2024 16:05
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
enhancement New feature or request tools integration Integration of swift-testing into tools/IDEs
Projects
None yet
Development

Successfully merging this pull request may close these issues.

__CommandLineArguments_v0 should default to JSON schema v0 rather than snapshot JSON
2 participants