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

depend on fissix, lib2to3 fork Fixes #202 #203

Merged
merged 6 commits into from
Jul 19, 2020
Merged

Conversation

graingert
Copy link
Member

No description provided.

README.rst Outdated Show resolved Hide resolved
@graingert
Copy link
Member Author

graingert commented May 13, 2020

while fissix doesn't support running under py2.7 or py3.5 it will still parse the code from those versions

@graingert
Copy link
Member Author

@takluyver ^

Copy link
Member

@jayvdb jayvdb left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

I've used this PR for rpm fixing @ https://build.opensuse.org/package/show/home:jayvdb:django/python-django-tdaemon .
Nice to have access to the extras of fissix.

@graingert
Copy link
Member Author

@takluyver is this package no longer maintained?

Copy link
Contributor

@takluyver takluyver left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Sorry - I've more or less finished using python-modernize, so there's usually something either more urgent or more interesting to pay attention to. I think this was more or less what happened before when Armin Rigo maintained it. It's prone to this, being mostly a one-off tool that people use and then move on.

Anyway, the fissix changes look OK to me; I've made a few comments inline.

Do you want to be a co-maintainer? CC @daira @brettcannon

libmodernize/fixes/fix_input_six.py Outdated Show resolved Hide resolved
libmodernize/fixes/fix_itertools_imports_six.py Outdated Show resolved Hide resolved
setup.py Show resolved Hide resolved
libmodernize/fixes/__init__.py Show resolved Hide resolved
LICENSE Outdated Show resolved Hide resolved
@graingert
Copy link
Member Author

Sorry - I've more or less finished using python-modernize
Do you want to be a co-maintainer? CC @daira @brettcannon

@takluyver it might be worth transferring the project to @jazzband

LICENSE Outdated Show resolved Hide resolved
@graingert
Copy link
Member Author

@takluyver I've addressed your review comments

@graingert graingert requested a review from takluyver July 19, 2020 15:21
@takluyver
Copy link
Contributor

Thanks, the changes look good. I'll do a squash merge to clean up the changes in comments that got reverted.

Jazzband is a neat idea, and I'd be open to it in principle. But I don't recognise most of the people involved (it looks like it's mostly come from the Django community, which I'm not familiar with), and python-modernize isn't my personal project, so I'd want to discuss with the other maintainers before transferring it. Whereas I recognise your name, and I'm happy enough to add another maintainer with just a heads-up to the other maintainers. So that's the easy option at the moment, if you're interested.

@takluyver takluyver merged commit decaf7c into PyCQA:master Jul 19, 2020
@graingert graingert deleted the fissix branch July 19, 2020 17:13
@graingert
Copy link
Member Author

@takluyver yeah that makes sense, I'd be happy to review PRs if you add me and I find some time

You might also like to invite @jreese

@graingert
Copy link
Member Author

@takluyver the @PyCQA is another place for group maintenance of Python packages (mainly focused on linting rather than Django)

@takluyver
Copy link
Contributor

I've sent you an invite. What's your PyPI username?- I can add you on there too. @jreese, I'm happy to invite you too if you're interested.

I'm also happy in principle to turn it over to PyCQA, but I'd want the other current maintainers to agree on a move like that. And presumably it also depends on whether the people in PyCQA consider this in scope.

@jayvdb
Copy link
Member

jayvdb commented Jul 19, 2020

fwiw, I am a maintainer at @jazzband. It does have a lot of Django stuff, but also pip-tools, prettytable, tablib, contextlib2 and others. If you wanted to transfer it under their wing, I think it would make sense for jazzband to find a way to expand their permission system and website so that the project can stay here, allowing organic growth of its own team and only falling back on jazzband where necessary.

@takluyver
Copy link
Contributor

Thanks @jayvdb. I'm not particularly concerned about the Github org - there's a single repo under the python-modernize org, and none of the maintainers have been very active recently, so I'd be happy enough to transfer it. But however it's technically done, I'd want @daira and @brettcannon to agree if some other group is going to adopt the codebase.

@graingert
Copy link
Member Author

@takluyver I'm pretty sure modernize fits under the category of code quality tools: here's the request from the bandit project: https://mail.python.org/pipermail/code-quality/2018-February/000976.html

@graingert
Copy link
Member Author

I've sent you an invite. What's your PyPI username?

I'm Thomas.Grainger on pypi

@takluyver
Copy link
Contributor

Thanks, I've added you on PyPI. I've also emailed Daira & Brett to ask about passing the project on to another group.

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