Skip to content

Fix license normalization type annotations. - #3112

Merged
freakboy3742 merged 1 commit into
beeware:mainfrom
filiplajszczak:typing/fix-license-config-annotations
Oct 27, 2026
Merged

freakboy3742 merged 1 commit into
beeware:mainfrom
filiplajszczak:typing/fix-license-config-annotations

Conversation

@filiplajszczak

@filiplajszczak filiplajszczak commented Oct 27, 2026 •

Copy link
Copy Markdown
Contributor

The license normalizers receive mutable merged TOML dictionaries, not AppConfig instances.

Annotate all five config parameters as dict[str, Any] and their return types as None, without changing runtime behavior.

Removes 15 ty diagnostics and leaves config.py clean with ty 0.0.89.

Refs #2709.

PR Checklist:

  • I will abide by the BeeWare Code of Conduct
  • I have read and have followed the CONTRIBUTING.md file
  • This PR was generated or assisted using an AI tool

Assisted-by: openai/gpt-6.1-sol

The license normalizers receive mutable merged TOML dictionaries, not AppConfig instances.

Annotate all five config parameters as dict[str, Any] and their return types as None, without changing runtime behavior.

Removes 15 ty diagnostics and leaves config.py clean with ty 0.0.89.

Refs beeware#2709.

@freakboy3742 freakboy3742 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.

👍 Thanks!

@freakboy3742
freakboy3742 merged commit 6517238 into beeware:main Oct 27, 2026
57 checks passed
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