Skip to content

Conversation

@thefeiter
Copy link

UnboundLocalError and UnboundFreeError use the term "variable" for something that is called a "name" in python. This can lead to confusion when getting this error.

This PR replaces the ocurance of "variable" with "name" in the error messagees of UnboundLocalError and UnboundFreeError.

UnboundLocalError uses the term "variable" for something that is called a "name" in python. This can lead to confusion when getting this error.
@ghost
Copy link

ghost commented Oct 18, 2023

All commit authors signed the Contributor License Agreement.
CLA signed

@bedevere-app
Copy link

bedevere-app bot commented Oct 18, 2023

Most changes to Python require a NEWS entry. Add one using the blurb_it web app or the blurb command-line tool.

If this change has little impact on Python users, wait for a maintainer to apply the skip news label instead.

@Eclips4
Copy link
Member

Eclips4 commented Oct 18, 2023

I'm -1 on this change.
As you can see, CI was failed due to failing tests, which depends on string representation of these exceptions.
It's also breaks backwards compatibility.

@bedevere-app
Copy link

bedevere-app bot commented Oct 18, 2023

Most changes to Python require a NEWS entry. Add one using the blurb_it web app or the blurb command-line tool.

If this change has little impact on Python users, wait for a maintainer to apply the skip news label instead.

@StanFromIreland StanFromIreland added the stale Stale PR or inactive for long period of time. label Oct 3, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

awaiting review stale Stale PR or inactive for long period of time.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants