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

Partial #6153 #4463 : reduce ipython_genutils usage #6214

Open
wants to merge 3 commits into
base: 6.4.x
Choose a base branch
from

Conversation

penguinolog
Copy link
Contributor

  • remove stdlib code can be used directly or helper redundant:
      * str is already unicode
      * type casting is done in external lib
      * import aliases
  • remove unused imports
  • reduce amount of deprecated code:
    • u prefix for strings
    • Logger.warn

`u` prefix was used in python 2, before all strings became unicode
* remove stdlib code can be used directly or helper redundant:
  * str is already unicode
  * type casting is done in external lib
  * import aliases
* remove unused imports
@penguinolog
Copy link
Contributor Author

(macos, 3.6) CI is configured incorrectly (CI issue)

@penguinolog
Copy link
Contributor Author

In case of python 3.6 kill it's possible wider migration with less copy-paste

@penguinolog
Copy link
Contributor Author

@kevin-bates Can I drop python 3.6 support and use more Ipython shared code?

@kevin-bates
Copy link
Member

Can I drop python 3.6 support and use more Ipython shared code?

I think dropping 3.6 is the right approach. Not sure what you mean by using "more ipython shared code" however. I thought this exercise was about removing ipython dependencies. Can you please provide examples of such shared code?

@penguinolog
Copy link
Contributor Author

Can I drop python 3.6 support and use more Ipython shared code?

I think dropping 3.6 is the right approach. Not sure what you mean by using "more ipython shared code" however. I thought this exercise was about removing ipython dependencies. Can you please provide examples of such shared code?

Main final goal is dropping ipython_genutils as having problems with modern python and officially recommended to not use.
BTW it's better to make this PR again from fresh branch after #6260 due to large diff

@kevin-bates
Copy link
Member

Main final goal is dropping ipython_genutils as having problems with modern python and officially recommended to not use.

I agree. You had used "ipython" in a previous response when we should try to replace with native python functions and why I needed some clarification. Thank you.

BTW it's better to make this PR again from fresh branch after #6260 due to large diff

Yes, I realized that as well last night.

@Zsailer Zsailer changed the base branch from main to 6.4.x March 7, 2022 18:20
@echarles echarles added this to the 6.4 milestone Mar 20, 2022
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.

3 participants