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

Skip building generated Github APIs in compat test #290

Merged

Conversation

simonjbeaumont
Copy link
Collaborator

@simonjbeaumont simonjbeaumont commented Sep 21, 2023

Motivation

We have an opt-in CI pipeline configured that is triggered by a comment. It runs the same compatibility test that's run in the automatic pipeline, which generates code for a set of real-world OpenAPI documents, but, additionally, it builds the generated code.

However, the Github APIs are so large that we cannot compile them in a reasonable amount of time in CI, if we want to compile the others.

Modifications

Skip building the generated code for Github APIs, even on the opt-in pipeline.

Result

The opt-in pipeline now runs in ~22 minutes.

Test Plan

We'll use this PR to trigger the CI to confirm it completes, and in a reasonable time.

@simonjbeaumont
Copy link
Collaborator Author

@swift-server-bot compat-test-full

@simonjbeaumont
Copy link
Collaborator Author

Hm... There are two compilations that got SIGKILLED... 🤔 https://ci.swiftserver.group/job/swift-openapi-generator-compatibility-test-full-prb/6/console

@simonjbeaumont
Copy link
Collaborator Author

@swift-server-bot compat-test-full

2 similar comments
@yim-lee
Copy link
Member

yim-lee commented Sep 27, 2023

@swift-server-bot compat-test-full

@yim-lee
Copy link
Member

yim-lee commented Sep 27, 2023

@swift-server-bot compat-test-full

@simonjbeaumont simonjbeaumont marked this pull request as ready for review September 28, 2023 09:31
@simonjbeaumont simonjbeaumont merged commit 6db47f7 into apple:main Sep 28, 2023
@czechboy0 czechboy0 added the semver/none No version bump required. label Oct 6, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
semver/none No version bump required.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants