Skip to content

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