Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

gh-64019: Add missing module attribute in inspect table #98116

Open
wants to merge 2 commits into
base: main
Choose a base branch
from

Conversation

slateny
Copy link
Contributor

@slateny slateny commented Oct 9, 2022

Copy link
Member

@JelleZijlstra JelleZijlstra left a comment

Looks good.

@ericsnowcurrently would you like to take a look since you reviewed the previous version of this PR?

Copy link
Member

@ericsnowcurrently ericsnowcurrently left a comment

LGTM

However, before merging, I recommend you get the opinion of @warsaw and @brettcannon; they are currently working on removing some of these attributes.

Copy link
Member

@AA-Turner AA-Turner left a comment

I think #65961 is the correct issue for the deprecations/removals.

A

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

6 participants