GV

GeezerVison

The World Wide Web, restored to 800 pixels of glory.

Home|About|Gaming cabinet|Archives|How it works|Starter sites

On this site

  • Sign in
  • ActionsCI/CD and automation
  • CodespacesCloud development environments
  • All Features
  • Documentation
  • Training
  • Certifications
  • Build with Copilot
  • Customize Copilot
  • Integrations
  • Trending repos
  • GitHub Sponsors
  • Developer articles
  • Enterprises

This view

  • Original page
  • New address

GeezerVison is showing github.com — gh-93963: Remove deprecated names from importlib.abc. by jaraco · Pull Request #94528 · python/cpython

Conversation

@jaraco

@jaraco jaraco commented Jul 3, 2022 •
edited
Loading

Uh oh!

There was an error while loading. Please reload this page.

Copy link
Copy Markdown
Member
  • Officially deprecate and remove abcs in importlib.abc moved to importlib.resources. #93963: Remove deprecated names from importlib.abc

Sorry, something went wrong.

Uh oh!

There was an error while loading. Please reload this page.

All reactions
jaraco and others added 5 commits June 17, 2022 16:23
@jaraco
gh-93963: Officially deprecate abcs and warn about their usage.
5e7fb9c
@jaraco @graingert
Adopt better wording
63c3c76
Co-authored-by: Thomas Grainger <[email protected]>
@jaraco
Use 'warnings._deprecated'. Fixes #93963 until Python π.
8525926
@jaraco
Remove generalization of a 'canonical' module and just repeat one's self
529e6ea
@jaraco @graingert
Identify the attribute name, not just the module in which it appears 🤦
908c96a
Co-authored-by: Thomas Grainger <[email protected]>
@jaraco jaraco added the DO-NOT-MERGE label Jul 3, 2022
@bedevere-bot bedevere-bot added the awaiting core review label Jul 3, 2022
@jaraco
jaraco changed the base branch from main to gh-93963/deprecate-importlib-resources-abcs July 3, 2022 14:51
@jaraco jaraco mentioned this pull request Jul 3, 2022
Officially deprecate and remove abcs in importlib.abc moved to importlib.resources. #93963
Closed
4 of 5 tasks
@jaraco
gh-93963: Remove deprecated names from importlib.abc
96f14c2
@jaraco
jaraco force-pushed the gh-93963/remove-importlib-resources-abcs branch from 3574e94 to 96f14c2 Compare July 3, 2022 14:58
This was referenced Jul 3, 2022
Bedevere rejects change that targets another PR python/bedevere#381
Closed
gh-93963: Officially deprecate abcs and warn about their usage. #93965
Merged
Base automatically changed from gh-93963/deprecate-importlib-resources-abcs to main July 3, 2022 19:17
@jaraco
Merge branch 'main' into gh-93963/remove-importlib-resources-abcs
691a82a
@jaraco

jaraco commented Jul 24, 2022

Copy link
Copy Markdown
Member Author

I see now in 691a82a, Github did the wrong thing. Apparently, because this repo uses squash merge, thereby rewriting history, Git can't tell that the lines removed in 96f14c2 were to be removed, and instead of complaining that there's a conflict, it just undid that change.

All reactions

Sorry, something went wrong.

Uh oh!

There was an error while loading. Please reload this page.

@jaraco
Remove the code wrongly restored in 96f14c2
17f614e
@jaraco

jaraco commented Jul 24, 2022 •
edited
Loading

Uh oh!

There was an error while loading. Please reload this page.

Copy link
Copy Markdown
Member Author

I've filed a Github support ticket to report the issue to Github and get their opinion on it. In the meantime, 17f614e should work around the issue.

All reactions

Sorry, something went wrong.

Uh oh!

There was an error while loading. Please reload this page.

@warsaw

warsaw commented Jul 24, 2022

Copy link
Copy Markdown
Member

I've filed a Github support ticket

Looks like probably only you @jaraco can see that ticket.

jaraco reacted with thumbs up emoji
All reactions
  • 👍 1 reaction

Sorry, something went wrong.

Uh oh!

There was an error while loading. Please reload this page.

@warsaw

warsaw commented Jul 24, 2022

Copy link
Copy Markdown
Member

@jaraco I'm trying to reboot myself on this discussion. @jaraco was the deprecation of access to these ABCs ever documented as deprecated in the library docs? I can't find that. That's not to say that it should hold up removal of these, since the deprecation warnings happened in the code. If we go ahead and remove the b/c hack now, I do think that any documentation update to importlib.resources.abc should explicitly mention the deprecation and removal history of access through importlib.abc.

All reactions

Sorry, something went wrong.

Uh oh!

There was an error while loading. Please reload this page.

@jaraco

jaraco commented Jul 29, 2022 •
edited
Loading

Uh oh!

There was an error while loading. Please reload this page.

Copy link
Copy Markdown
Member Author

Looks like probably only you @jaraco can see that ticket.

Yeah, that's true. That's how Github does things. I linked it here to be transparent about what I'm doing and to associate a reference. Still no response, BTW.

was the deprecation of access to these ABCs ever documented as deprecated in the library docs?

I did not previously include that effort as part of this change because Hugo and others were working on docs updates. The deprecations were subsequently documented in #94546.

All reactions

Sorry, something went wrong.

Uh oh!

There was an error while loading. Please reload this page.

@jaraco
Merge branch 'main' into gh-93963/remove-importlib-resources-abcs
9cb1fc4
@jaraco jaraco mentioned this pull request Aug 20, 2022
Recommend a procedure for branch management python/core-workflow#475
Closed
@CAM-Gerlach

CAM-Gerlach commented Dec 15, 2022 •
edited
Loading

Uh oh!

There was an error while loading. Please reload this page.

Copy link
Copy Markdown
Member

Hey @jaraco , what are the next steps here?

Just FYI, following what was discussed in python/core-workflow#475 , this is the only personal feature branch remaining on the upstream cpython repo, which blocks the full implementation of the branch protection rule approved in python/steering-council#144 that would avoid the issue where it is easy to push personal feature branches to the upstream by accident, either through the Git CLI or via GitHub's web Edit feature.

All reactions

Sorry, something went wrong.

Uh oh!

There was an error while loading. Please reload this page.

@CAM-Gerlach CAM-Gerlach mentioned this pull request Dec 15, 2022
Adjusting the "Triager" role's permissions python/core-workflow#460
Closed
@jaraco

jaraco commented Dec 17, 2022

Copy link
Copy Markdown
Member Author

Let's close this and delete the branch and hope someone can find the changes when the time comes to submit it.

CAM-Gerlach reacted with thumbs up emoji
All reactions
  • 👍 1 reaction

Sorry, something went wrong.

Uh oh!

There was an error while loading. Please reload this page.

@jaraco jaraco closed this Dec 17, 2022
@jaraco
jaraco deleted the gh-93963/remove-importlib-resources-abcs branch December 17, 2022 16:24
@hugovk hugovk mentioned this pull request May 29, 2024
gh-93963: Remove deprecated names from importlib.abc #119720
Merged
@hugovk

hugovk commented May 29, 2024

Copy link
Copy Markdown
Member

Re-done as #119720.

All reactions

Sorry, something went wrong.

Uh oh!

There was an error while loading. Please reload this page.

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

Reviewers

No reviews

Uh oh!

There was an error while loading. Please reload this page.

Assignees

No one assigned

Labels

awaiting core review DO-NOT-MERGE

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

Uh oh!

There was an error while loading. Please reload this page.

5 participants

@jaraco @warsaw @CAM-Gerlach @hugovk @bedevere-bot
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
You are visitor 01402503 — GeezerVison does not store the pages you look up.
Best viewed with Netscape Navigator 4.7 or Internet Explorer 6 at 800×600. No cookies required.