Skip to content

[build-presets] Skip libdispatch tests for XCTest #4942

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

Conversation

modocache
Copy link
Contributor

The corelibs-xctest preset is one that swift-corelibs-xctest contributors could use locally to quickly ensure that XCTest builds and its tests pass. The libdispatch tests add a significant amount of time to this preset, so remove them.

The `corelibs-xctest` preset is one that swift-corelibs-xctest
contributors could use locally to quickly ensure that XCTest builds and its
tests pass. The libdispatch tests add a significant amount of time to
this preset, so remove them.
@modocache
Copy link
Contributor Author

@swift-ci please smoke test and merge

@modocache
Copy link
Contributor Author

Looks like an unrelated build failure on macOS, due to a stale build of CoreImage on the CI server.

@modocache
Copy link
Contributor Author

@swift-ci please test macOS

@modocache
Copy link
Contributor Author

modocache commented Sep 23, 2016

@shahmishal Oops! I think "please test macOS" above triggered a "Test and Merge (smoke test)" CI job. Is this a bug?

@swift-ci
Copy link
Contributor

Build failed
Jenkins build - Swift Test OS X Platform
Git Commit - be1df57
Test requested by - @modocache

@shahmishal
Copy link
Member

I think it was triggered due to this comment:

"modocache commented an hour ago
swift-ci please smoke test and merge"

@modocache
Copy link
Contributor Author

Oh, my bad. I was looking at two other pull requests I submitted and must have gotten confused! 😅

Sorry for taking your time!

@modocache
Copy link
Contributor Author

@swift-ci please smoke test

@modocache
Copy link
Contributor Author

This has zero impact on CI, so I'm going to go ahead and merge.

@modocache modocache merged commit 229beba into swiftlang:master Sep 25, 2016
@modocache modocache deleted the preset-corelibs-xctest-skip-test-libdispatch branch September 25, 2016 20:12
MaxDesiatov pushed a commit that referenced this pull request Sep 7, 2023
[pull] swiftwasm from main
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants