Skip to content

networkx: Fix incompletes in connectivity/connectivity#14661

Merged
srittau merged 9 commits intopython:mainfrom
MeGaGiGaGon:networkx-update-connectivity
Aug 31, 2025
Merged

networkx: Fix incompletes in connectivity/connectivity#14661
srittau merged 9 commits intopython:mainfrom
MeGaGiGaGon:networkx-update-connectivity

Conversation

@MeGaGiGaGon
Copy link
Copy Markdown
Contributor

@MeGaGiGaGon MeGaGiGaGon commented Aug 29, 2025

Adds type hints to previously incomplete items in algorithms/connectivity/connectivity.pyi. Changes were done referencing https://networkx.org/documentation/stable/_modules/networkx/algorithms/connectivity/connectivity.html

@MeGaGiGaGon
Copy link
Copy Markdown
Contributor Author

I realized what I did wrong, there are two files named connectivity.pyi in networkx with almost the same functions and I was editing the wrong one 🤦

@MeGaGiGaGon MeGaGiGaGon changed the title networkx: Add new things to connectivity networkx: Fix incompletes in connectivity/connectivity Aug 29, 2025
@github-actions

This comment has been minimized.

@github-actions
Copy link
Copy Markdown
Contributor

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

discord.py (https://github.com/Rapptz/discord.py)
- discord/ext/commands/hybrid.py:508: error: Overlap between argument names and ** TypedDict items: "description", "name"  [misc]
+ discord/ext/commands/hybrid.py:508: error: Overlap between argument names and ** TypedDict items: "name", "description"  [misc]
- discord/ext/commands/hybrid.py:629: error: Overlap between argument names and ** TypedDict items: "description", "name"  [misc]
+ discord/ext/commands/hybrid.py:629: error: Overlap between argument names and ** TypedDict items: "name", "description"  [misc]

@srittau srittau merged commit 2480d7e into python:main Aug 31, 2025
48 checks passed
@MeGaGiGaGon MeGaGiGaGon deleted the networkx-update-connectivity branch September 1, 2025 00:15
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