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

Active fork of grumpy #68

Closed
alanjds opened this issue Aug 21, 2018 · 20 comments
Closed

Active fork of grumpy #68

alanjds opened this issue Aug 21, 2018 · 20 comments
Labels
imported Imported from google/grumpy

Comments

@alanjds
Copy link

alanjds commented Aug 21, 2018

google#406 opened by @funny-falcon on Jul 4

Looks like @alanjds maintains active fork of this project https://github.com/alanjds/grumpy
Lets support him with our attention!

@alanjds
Copy link
Author

alanjds commented Aug 21, 2018

Comment by cclauss
Wednesday Jul 04, 2018 at 11:14 GMT


I am highly supportive of this effort but...

  1. We can not open new issues at the fork. Should we just open and track issues here?
  2. Current course and speed, will this this project succeed at just the point where Python 2 reaches end of life?

@alanjds
Copy link
Author

alanjds commented Aug 21, 2018

Comment by corona10
Wednesday Jul 04, 2018 at 11:42 GMT


There are 2 ways

  1. Lets start from base to support python3.
  2. Aim this project to archiving python2 code

Anyway I would like to maintain or cowork this project as a new grumpy org not someone's repository.

@alanjds
Copy link
Author

alanjds commented Aug 21, 2018

Comment by corona10
Wednesday Jul 04, 2018 at 11:58 GMT


@alanjds WDYT?

@alanjds
Copy link
Author

alanjds commented Aug 21, 2018

Comment by alanjds
Wednesday Jul 04, 2018 at 19:06 GMT


Hi.

First of all, thanks for you support. I am preparing to publish an article detailing the new features of my branch. If you can, please leave comments before I do release it next week: https://labs.getninjas.com.br/unfinished-draft-released-grumpy-runtime-v0-3-0-a05f1cf8e111

@corona10:

  1. I think its better to keep the existing codebase, but build over Python3 support.
  2. AND keep aiming for Python2 support.

In fact is not hard to support both for now. I do think about a command line switch, or even try to parse as Py3 and fallback to Py2. But the way to do it is not important now I guess. Better is to have the only Python 2-and-3 compatible runtime and maybe let projects mix both flavors to ease the upgrade to Py3.

When messing with the compiler, made clear to me that many Py3 features are just compatible with Py2, and as easy or easier to implement. For exemple the PEP-3174 (__pycache__ folders) is so nicer then trying to create .pyc files.

@cclauss:

  1. Fixed :) You can create issues there now. But should we deprecate this google/grumpy repo?
  2. Not with just me coding ;-]

In fact, personally I am committed to push Grumpy until Python 2.7 EOL. If it does not be close to substitute Python 2.7 there, I will see it as sink cost of my bandwith and stop.

The whole work I made in this time after forking is all about making easier/appealing for new people to contribute. Now the job will be of getting more hands to help. Please take a look at the article I linked up. I do think it is a lot more welcoming than before, but maybe not enough.

Anyway I would like to maintain or cowork this project as a new grumpy org not someone's repository.

I had hold myself from a hard-hard fork until have something useful/better to show. And this repo here have 9k starts that will not be pulled over. Was hoping Google to create the organization and hand it over (what you think @trotterdylan ?). But is time to do the new organization :/

We may create a whole new organization, like grumpyorg/grumpy, or put it on https://jazzband.co one.

What do you think?

@alanjds
Copy link
Author

alanjds commented Aug 21, 2018

Comment by corona10
Wednesday Jul 04, 2018 at 23:24 GMT


@alanjds

Yes, is time to create a grumpy.org / grumpy.

Few things should be discussed.

  1. I know your branch has a head from 198 commits from google / grumpy, as you know, https://github.com/google/grumpy/pulls?q=is%3Apr+is%3Aclosed we merged all commits per feature units. So would you like to squash your 198 commits into few feature based commits.

  2. IMHO, we don't need 9k stars this is not a facebook, I mean we should focus on maintaining source codes not stars.

  3. We should move all issues to new org issues.

  4. grumpy project has highly dependent on https://github.com/m-labs/pythonparser to parse Python code. we should fork this one also into grumpy.org / pythonparser .

Please understand my short English thanks!

@alanjds
Copy link
Author

alanjds commented Aug 21, 2018

Comment by corona10
Wednesday Jul 04, 2018 at 23:44 GMT


@alanjds
I saw one case that tries transfer into the new repository but failed.
pyston/pyston_v1#1422

@alanjds
Copy link
Author

alanjds commented Aug 21, 2018

Comment by alanjds
Thursday Jul 05, 2018 at 00:06 GMT


Your english is damn good for me.

  1. Maybe. I am not a fan of squashing commits because it makes bisecting very harder, but maybe for a one-off... Lots of new features are dependent on each other. I had not removed the branch marks on Github, so would be easy to redo the paths, but is it worth?
  2. Agreed, but, eh... its all about more hands. The 398 watchers are being notified as we talk and this is a nice asset. I am ok to lose it, but prefer not if possible.
  3. Totally agreed.
  4. Right now the pythonparser is vendored inside Grumpy repo. I thought about unvendoring it and search for a working "stock" version. This will help with the Py3 quest too. May be needed to fork it, but I vote for trying some "stock" versions first.

Sad on pyston repo :/ Corporates sometimes have odd decisions. At least Google is not Facebook nor Oracle and Grumpy is Apache 2 already. Thanks for all the code I could not write myself 😘.

@alanjds
Copy link
Author

alanjds commented Aug 21, 2018

Comment by alanjds
Sunday Jul 08, 2018 at 17:27 GMT


Yesterday I published grumpy-tools v0.2.1, and now all the article examples do work.

I want to public publish the article this week.

However there is one feature on the paths branch that I do not know if should be included: it includes the whole sys.path on the search for imports. Means that everything pip-installed is available to be transpiled.

Should it be included?

@alanjds
Copy link
Author

alanjds commented Aug 21, 2018

Comment by alanjds
Sunday Jul 08, 2018 at 17:30 GMT


Other thing: will we create the grumpyorg to include in the article or publish another later pointing people to it?

Have you a preference on a new organization or jazzband?

@alanjds
Copy link
Author

alanjds commented Aug 21, 2018

Comment by alanjds
Tuesday Jul 10, 2018 at 17:17 GMT


Merged the paths branch and released grumpy-runtime 0.2.2 on PyPI. Should be up in a few minutes.

If nothing new cames, I should release the article Tuesday 17.

@alanjds
Copy link
Author

alanjds commented Aug 21, 2018

Comment by alanjds
Sunday Jul 22, 2018 at 17:41 GMT


@corona10

Have you thought about moving the fork to a new org or to jazzband one?

I am ready to release the 0.3.0 version and publish the article, and want to migrate the issues before that if possible.

@alanjds
Copy link
Author

alanjds commented Aug 21, 2018

Comment by RezaGhanbari
Wednesday Jul 25, 2018 at 07:31 GMT


@alanjds I think it's better to move this active fork to another organization with python 3 support.

@alanjds
Copy link
Author

alanjds commented Aug 21, 2018

Comment by alanjds
Wednesday Jul 25, 2018 at 16:51 GMT


@rezaghanbari Ok. Will create a new organization instead of jazzband, and will invite all you 🙈 ❤️

Python 3 support is a matter of listing the features and start implementing. Most missing stuff is needed both on Python2 and Python3:

Right now I am on the quest to support import *, needed to be able to run grumpy -m pip and grumpy -m httpie.

Btw, is released the grumpy-runtime v0.3.0 on PyPI. The article explaining the new features on my branch is released too: https://labs.getninjas.com.br/released-grumpy-runtime-v0-3-0-a05f1cf8e111

@alanjds
Copy link
Author

alanjds commented Aug 21, 2018

Comment by corona10
Thursday Jul 26, 2018 at 05:11 GMT


@alanjds
I agree with new org, not jazzband.
IMHO, There is no reason to use jazzband

Anyway congrats to release the new version 👍

@alanjds
Copy link
Author

alanjds commented Aug 21, 2018

Comment by alanjds
Saturday Jul 28, 2018 at 14:01 GMT


Huh, @grumpyorg is now taken. I saw it available last week. What will be the new name now?

@alanjds
Copy link
Author

alanjds commented Aug 21, 2018

Comment by RezaGhanbari
Saturday Jul 28, 2018 at 14:05 GMT


@alanjds what about combination of grumpier and org?
Or just grumpier?

@alanjds
Copy link
Author

alanjds commented Aug 21, 2018

Comment by alanjds
Saturday Jul 28, 2018 at 14:12 GMT


Taken too :/

@alanjds
Copy link
Author

alanjds commented Aug 21, 2018

Comment by alanjds
Saturday Jul 28, 2018 at 14:13 GMT


grumpyhome is available

@alanjds
Copy link
Author

alanjds commented Aug 21, 2018

Comment by alanjds
Saturday Jul 28, 2018 at 14:18 GMT


The new grumpy home is at https://github.com/grumpyhome/grumpy !

@alanjds alanjds added the imported Imported from google/grumpy label Aug 21, 2018
@alanjds
Copy link
Author

alanjds commented Sep 2, 2018

Closing, as the new home is now here. Thanks @funny-falcon and others for the good feedback and motivation to keep going.

@alanjds alanjds closed this as completed Sep 2, 2018
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
imported Imported from google/grumpy
Projects
None yet
Development

No branches or pull requests

1 participant