Skip to content

fix: include stdout in swiftpm failure message#2245

Merged
mergify[bot] merged 2 commits into
cgrindel:mainfrom
keith:ks/include-stdout-in-swiftpm-failure-message
May 3, 2026
Merged

fix: include stdout in swiftpm failure message#2245
mergify[bot] merged 2 commits into
cgrindel:mainfrom
keith:ks/include-stdout-in-swiftpm-failure-message

Conversation

@keith
Copy link
Copy Markdown
Contributor

@keith keith commented Apr 30, 2026

Swiftly prints some errors to stdout:

`...external/rules_swift_package_manager++swift_deps+swiftpkg_rxswift/.swift-version`
uses toolchain version 6.1, but it doesn't match any of the installed
toolchains. You can install the toolchain with `swiftly install`.

Swiftly prints some errors to stdout:

```
`/home/ksmiley/.cache/bazel/_bazel_ksmiley/ece89d9580e3e8875ad626876be0ba4b/external/rules_swift_package_manager++swift_deps+swiftpkg_rxswift/.swift-version`
uses toolchain version 6.1, but it doesn't match any of the installed
toolchains. You can install the toolchain with `swiftly install`.
```
@keith
Copy link
Copy Markdown
Contributor Author

keith commented Apr 30, 2026

swiftlang/swiftly#531

@cgrindel cgrindel changed the title Include stdout in swiftpm failure message fix: include stdout in swiftpm failure message May 2, 2026
@cgrindel
Copy link
Copy Markdown
Owner

cgrindel commented May 2, 2026

@Mergifyio queue

@mergify
Copy link
Copy Markdown
Contributor

mergify Bot commented May 2, 2026

Merge Queue Status

  • Entered queue2026-05-02 15:35 UTC · Rule: default
  • Checks passed · in-place
  • Merged2026-05-03 12:45 UTC · at efee622b03cd169b0f8897e4e69ad77bc5ea25b1

This pull request spent 21 hours 10 minutes 19 seconds in the queue, including 19 hours 11 minutes 27 seconds running CI.

Required conditions to merge

@mergify mergify Bot added the queued label May 2, 2026
@mergify mergify Bot merged commit f6b575a into cgrindel:main May 3, 2026
42 checks passed
@mergify mergify Bot removed the queued label May 3, 2026
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