Skip to content

Publish patched rules_swift as codeql.2 - #22416

Merged
jketema merged 1 commit into
mainfrom
jketema/swift-static-versioning
Aug 24, 2026
Merged

Publish patched rules_swift as codeql.2#22416
jketema merged 1 commit into
mainfrom
jketema/swift-static-versioning

Conversation

@jketema

@jketema jketema commented Aug 24, 2026

Copy link
Copy Markdown
Contributor

Failures expected here. Depends on internal PR.

Co-authored-by: Copilot 223556219+Copilot@users.noreply.github.com

Co-authored-by: Copilot <223556219+Copilot@users.noreply.github.com>

Copilot-Session: 7b8af11e-692d-4c94-87be-8cdaaf908e7c
@jketema
jketema marked this pull request as ready for review August 24, 2026 14:31
@jketema
jketema requested review from a team as code owners August 24, 2026 14:31
Copilot AI balanced review requested due to automatic review settings August 24, 2026 14:31

Copilot AI left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Pull request overview

Publishes the patched rules_swift dependency as 4.0.0-rc5-codeql.2.

Changes:

  • Updates the repository to use the new patched version.
  • Adds macOS downloaded-toolchain support.
  • Adds external static-runtime and runtime-file support.
Show a summary per file
File Description
MODULE.bazel Selects rules_swift codeql.2.
misc/bazel/registry/modules/rules_swift/metadata.json Registers the new version.
misc/bazel/registry/modules/rules_swift/4.0.0-rc5-codeql.2/source.json Defines upstream source and patches.
misc/bazel/registry/modules/rules_swift/4.0.0-rc5-codeql.2/patches/register_downloaded_macos_toolchain.patch Configures downloaded macOS toolchains.
misc/bazel/registry/modules/rules_swift/4.0.0-rc5-codeql.2/patches/external_static_runtime.patch Adds external runtime integration.
misc/bazel/registry/modules/rules_swift/4.0.0-rc5-codeql.2/MODULE.bazel Declares the patched module version.

Review details

💡 Add a code-review agent skill or configure MCP servers for context-aware, tailored reviews. Learn more in the docs.

  • Files reviewed: 3/6 changed files
  • Comments generated: 0
  • Review effort level: Balanced

@jketema jketema added the depends on internal PR This PR should only be merged in sync with an internal Semmle PR label Aug 24, 2026
@jketema
jketema merged commit 9a34fbc into main Aug 24, 2026
24 of 82 checks passed
@jketema
jketema deleted the jketema/swift-static-versioning branch August 24, 2026 14:55
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

depends on internal PR This PR should only be merged in sync with an internal Semmle PR

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants