Skip to content

Move JSON ABI source files to prepare for v1. #953

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 2 commits into from
Feb 12, 2025

Conversation

grynspan
Copy link
Contributor

@grynspan grynspan commented Feb 12, 2025

At this time, we expect most of the "ABIv1" JSON schema will be the same as the current "ABIv0" one, so this PR reshuffles the source files supporting it so they can be more readily reused between the two.

Future PRs will start adding actual "ABIv1" functionality—this is just a bookkeeping PR.

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 the tools integration 🛠️ Integration of swift-testing into tools/IDEs label Feb 12, 2025
@grynspan grynspan added this to the Swift 6.x milestone Feb 12, 2025
@grynspan grynspan self-assigned this Feb 12, 2025
At this time, we expect most of the "ABIv1" JSON schema will be the same as the
current "ABIv0" one, so this PR reshuffles the source files supporting it so
they can be more readily reused between the two.

Future PRs will start adding actual "ABIv1" functionality—this is just a
bookkeeping PR.
@grynspan grynspan force-pushed the jgrynspan/reshuffle-abi-sources branch from 8bd792b to c988c2b Compare February 12, 2025 17:53
@grynspan grynspan marked this pull request as ready for review February 12, 2025 18:32
@grynspan
Copy link
Contributor Author

@swift-ci test

@grynspan
Copy link
Contributor Author

@swift-ci test

@grynspan grynspan merged commit 7493d6e into main Feb 12, 2025
3 checks passed
@grynspan grynspan deleted the jgrynspan/reshuffle-abi-sources branch February 12, 2025 21:30
Copy link
Contributor

@stmontgomery stmontgomery left a comment

Choose a reason for hiding this comment

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

LGTM thanks!

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
tools integration 🛠️ Integration of swift-testing into tools/IDEs
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants