Skip to content

[tests] Run swift-(corelibs-)foundation tests in debug configuration on Windows - #80140

Merged
shahmishal merged 1 commit into
swiftlang:release/6.1from
ahoppen:6.1/foundation-test-debug
Mar 20, 2025
Merged

shahmishal merged 1 commit into
swiftlang:release/6.1from
ahoppen:6.1/foundation-test-debug

Conversation

@ahoppen

@ahoppen ahoppen commented Mar 19, 2025

Copy link
Copy Markdown
Member
  • Explanation: This should improve CI times because building in debug configuration instead of release configuration is significantly faster. Since we don’t install the build of swift-(corelibs-)foundation using SwiftPM into the toolchain, this doesn’t have any performance impact on users of the toolchain.
  • Scope: Testing of swift-(corelibs-)foundation, no impact on any products installed into the toolchain
  • Issue: n/a
  • Original PR: [tests] Run swift-(corelibs-)foundation tests in debug configuration on Windows #80076
  • Risk: Very low, since this doesn’t affect any binaries installed into the toolchain
  • Testing: Verified that testing is about 35 minutes faster
  • Reviewer: @jmschonfeld

…on Windows

This should improve CI times because building in debug configuration instead of release configuration is significantly faster. Since we don’t install the build of swift-(corelibs-)foundation using SwiftPM into the toolchain, this doesn’t have any performance impact on users of the toolchain.
@ahoppen
ahoppen requested a review from a team as a code owner March 19, 2025 20:34
@ahoppen

ahoppen commented Mar 19, 2025

Copy link
Copy Markdown
Member Author

@swift-ci Please test

@shahmishal
shahmishal enabled auto-merge March 19, 2025 22:26
@ahoppen

ahoppen commented Mar 20, 2025

Copy link
Copy Markdown
Member Author

@swift-ci Please test Windows

@shahmishal
shahmishal merged commit 95c460d into swiftlang:release/6.1 Mar 20, 2025
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.

2 participants