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

Jump to bottom

Swift: rework workflows #11008

Draft
wants to merge 2 commits into
base: main
Choose a base branch
from
Draft

Swift: rework workflows #11008

wants to merge 2 commits into from
+205 −175

Conversation

redsun82
Copy link
Contributor

@redsun82 redsun82 commented Oct 27, 2022

No description provided.

@redsun82 redsun82 force-pushed the redsun82/swift-macos-integration-tests branch 3 times, most recently from 71f4a6c to cb887a2 Compare Oct 27, 2022
@redsun82 redsun82 changed the title Swift: enable integration tests on macOS Swift: rework workflows Oct 27, 2022
@redsun82 redsun82 force-pushed the redsun82/swift-macos-integration-tests branch 2 times, most recently from 7383d88 to 0b6695a Compare Oct 27, 2022
@github-actions github-actions bot added the Swift label Oct 27, 2022
@redsun82 redsun82 force-pushed the redsun82/swift-macos-integration-tests branch 11 times, most recently from 836b0dc to 2db2aa9 Compare Oct 28, 2022
@github-actions github-actions bot added Swift and removed Swift labels Oct 28, 2022
@redsun82 redsun82 force-pushed the redsun82/swift-macos-integration-tests branch 10 times, most recently from 977e02d to 41de3ec Compare Oct 31, 2022
redsun82 added 2 commits Oct 31, 2022
@redsun82
Swift: rework workflows
0038666 
* A unique workflow file has been created merging all `swift-*.yml`
  workflows
* Change filtering at job level was added using [dorny/paths-filter][1]
* only one build of the extractor is made, and then shared via cache
  (not as an artifact because of [this longstading issue][2])
* integration tests are now run on on macOS
* qltests are not run any more on macOS to cut on feedback time
* autobuilder tests were moved to the macOS build step to avoid loading
  bazel twice

[1]: https://github.com/dorny/paths-filter#examples
[2]: actions/upload-artifact#38
@redsun82
Swift: use macos-12-xl runners
80ea2b8
@redsun82 redsun82 force-pushed the redsun82/swift-macos-integration-tests branch from 41de3ec to 80ea2b8 Compare Oct 31, 2022
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@github/codeql-c codeql-c

@github/codeql-ci-reviewers codeql-ci-reviewers

At least 1 approving review is required to merge this pull request.

Assignees
No one assigned
Labels
Swift
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.

None yet
1 participant
@redsun82