Skip to content

[ssl] Set values to constants - #14544

Merged
srittau merged 2 commits into
python:mainfrom
donbarbos:set-values-to-constants
Aug 8, 2025
Merged

srittau merged 2 commits into
python:mainfrom
donbarbos:set-values-to-constants

Conversation

@donbarbos

Copy link
Copy Markdown
Contributor

@github-actions

This comment has been minimized.

Comment thread stdlib/ssl.pyi Resolved
@github-actions

github-actions Bot commented Aug 8, 2025

Copy link
Copy Markdown
Contributor

Diff from mypy_primer, showing the effect of this PR on open source code:

urllib3 (https://github.com/urllib3/urllib3)
+ src/urllib3/util/ssl_.py:139: error: Cannot assign to final name "OP_NO_COMPRESSION"  [misc]
+ src/urllib3/util/ssl_.py:139: note: Error code "misc" not covered by "type: ignore" comment
+ src/urllib3/util/ssl_.py:140: error: Cannot assign to final name "OP_NO_TICKET"  [misc]
+ src/urllib3/util/ssl_.py:140: note: Error code "misc" not covered by "type: ignore" comment
+ src/urllib3/util/ssl_.py:141: error: Cannot assign to final name "OP_NO_SSLv2"  [misc]
+ src/urllib3/util/ssl_.py:141: note: Error code "misc" not covered by "type: ignore" comment
+ src/urllib3/util/ssl_.py:142: error: Cannot assign to final name "OP_NO_SSLv3"  [misc]
+ src/urllib3/util/ssl_.py:142: note: Error code "misc" not covered by "type: ignore" comment
+ src/urllib3/util/ssl_.py:143: error: Cannot assign to final name "PROTOCOL_TLS"  [misc]
+ src/urllib3/util/ssl_.py:143: note: Error code "misc" not covered by "type: ignore" comment
+ src/urllib3/util/ssl_.py:144: error: Cannot assign to final name "PROTOCOL_TLS_CLIENT"  [misc]
+ src/urllib3/util/ssl_.py:144: note: Error code "misc" not covered by "type: ignore" comment
+ src/urllib3/util/ssl_.py:146: error: Cannot assign to final name "VERIFY_X509_STRICT"  [misc]
+ src/urllib3/util/ssl_.py:146: note: Error code "misc" not covered by "type: ignore" comment

@srittau
srittau merged commit 56264ac into python:main Aug 8, 2025
@donbarbos
donbarbos deleted the set-values-to-constants branch February 25, 2026 04:22
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