Skip to content

[ssl] Set values to constants#14544

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

[ssl] Set values to constants#14544
srittau merged 2 commits intopython:mainfrom
donbarbos:set-values-to-constants

Conversation

@donbarbos
Copy link
Contributor

@github-actions

This comment has been minimized.

@github-actions
Copy link
Contributor

github-actions bot commented Aug 8, 2025

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
63 checks passed
@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