Skip to content

Restore feature_configuration argument to derive_swift_module_name - #1907

Merged
brentleyjones merged 1 commit into
bazelbuild:mainfrom
brentleyjones:bj/restore-module-name-feature-configuration
Oct 2, 2026
Merged

brentleyjones merged 1 commit into
bazelbuild:mainfrom
brentleyjones:bj/restore-module-name-feature-configuration

Conversation

@brentleyjones

@brentleyjones brentleyjones commented Sep 21, 2026 •

Copy link
Copy Markdown
Collaborator

Restore the optional feature_configuration keyword argument accepted by derive_swift_module_name in 4.0.1 and 4.1.0. Its removal in #1903 breaks existing callers even though the argument is currently unused.

Keep the argument as a no-op with an explicit unused-variable suppression, and regenerate the API documentation. This preserves compatibility for the next patch release.

@brentleyjones brentleyjones added the AI Generated Changes prepared with AI assistance label Sep 21, 2026
@brentleyjones
brentleyjones marked this pull request as ready for review September 21, 2026 14:47
@brentleyjones
brentleyjones enabled auto-merge (squash) September 21, 2026 14:48
Comment thread swift/module_name.bzl

@keith keith left a comment

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

harmless enough for now

@brentleyjones
brentleyjones merged commit d901fb8 into bazelbuild:main Oct 2, 2026
16 of 18 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

AI Generated Changes prepared with AI assistance

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants