Skip to content

Conversation

joesho112358
Copy link

Adding reverse method for strings

Broke format because I am not sure this will even be accepted and it is a draft at this point, so didn't want to have this and an issue open. Nevertheless, this is an attempt to add a .reverse() method for strings. I think most modern languages should have something like that and [::-1] is a bit archaic with little charm.

It is a draft because I haven't gotten it working and am looking for what I may have missed!

@ghost
Copy link

ghost commented Feb 20, 2025

All commit authors signed the Contributor License Agreement.
CLA signed

@bedevere-app
Copy link

bedevere-app bot commented Feb 20, 2025

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.

@bedevere-app
Copy link

bedevere-app bot commented Feb 20, 2025

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.

@ZeroIntensity
Copy link
Member

Sorry, there's a process we have to follow for features, especially for changes to builtins:

  • Bring this up on DPO first--this is to weed out bad ideas, or ones that lack motivation from the author.
  • If all goes well over there (which is difficult), you would normally create an issue, but changes to builtins require PEPs.
  • If you get a PEP written (which requires a core dev sponsor), you can send it to the SC and if they approve it, you can create an issue.
  • Only then, after all that process, you can make a PR.

@joesho112358
Copy link
Author

@ZeroIntensity thanks for that! Looks like i misinterpreted the devguide and missed this section! I will make an attempt in the ideas section there

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