Skip to content

More precise types in preserve_defaults.py - #12385

Merged
chrisjsewell merged 1 commit into
sphinx-doc:masterfrom
JelleZijlstra:patch-1
May 18, 2024
Merged

More precise types in preserve_defaults.py#12385
chrisjsewell merged 1 commit into
sphinx-doc:masterfrom
JelleZijlstra:patch-1

Conversation

@JelleZijlstra

Copy link
Copy Markdown
Contributor

Followup from python/typeshed#11880. Not all AST nodes have a lineno; in this file only parameter defaults are passed to this function, which are always ast.expr.

Subject: Prepare for future typing errors from a mypy upgrade

Feature or Bugfix

  • Refactoring

Followup from python/typeshed#11880. Not all AST nodes have a lineno; in this file only parameter defaults are passed to this function, which are always ast.expr.
@JelleZijlstra
JelleZijlstra marked this pull request as ready for review May 18, 2024 15:19

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

Cheers!

@chrisjsewell
chrisjsewell merged commit 6ce4f96 into sphinx-doc:master May 18, 2024
@JelleZijlstra
JelleZijlstra deleted the patch-1 branch May 18, 2024 20:02
@github-actions github-actions Bot locked as resolved and limited conversation to collaborators Jun 18, 2024
@AA-Turner AA-Turner added this to the 7.4.0 milestone Jul 13, 2024
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants