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

chore(py3): remove outdated python2 utf8 hint #207

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

Conversation

Rotzbua
Copy link
Contributor

@Rotzbua Rotzbua commented Oct 20, 2023

No description provided.

@svenevs
Copy link
Owner

svenevs commented Oct 22, 2023

This looks great, I want to wait on this or any py2 dead code elimination stuff. The next release is going to prioritize

  • Fixing the packaging requirements
  • Adding concepts
  • A few small issues that can be fixed with minimal effort

I want to do this with as few code changes as possible, with the hope that after releasing no flood of issues come in saying I broke something. If all goes well, then I will want to clean the crap out of this codebase and will start here 🙂

Just wanted to give you a heads up that while I really appreciate your help getting this all cleaned up, I want to be very careful about how I update what and when. There's a lot of ... shenanigans ... in graph.py in particular, and I worry about doing a release that adds new features and does huge backend changes.

While I don't see any reason why the changes here would cause problems, I want the next release to have as small a diff as possible to make it easier to bisect if the new features actually break something.

@Rotzbua Rotzbua force-pushed the migrate_code_utf branch from 9572356 to 95225f8 Compare May 15, 2024 23:48
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