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

More maintainers? #32

Closed
reedy opened this issue May 13, 2020 · 15 comments
Closed

More maintainers? #32

reedy opened this issue May 13, 2020 · 15 comments

Comments

@reedy
Copy link

reedy commented May 13, 2020

While I'm glad this was forked into an org, so that if the current maintainers don't want to do it anymore, they can easily add more (vs just being in a users namespace) maintainers etc.

It does seem like the current maintainers aren't that actually that active (which is certainly understandable given COVID-19), but it's slightly concerning when they've only recently taken over the project.

To that extent, can we get more people involved to help this? Numerous people are still waiting on various cleanup from the forking and renaming, to deal with the deprecated/abandoned warnings when running composer.

Originally filed at php-parallel-lint/PHP-Console-Highlighter#9 but I'm not sure why I put it in there

@legoktm
Copy link

legoktm commented May 21, 2020

+1, at @wikimedia we rely pretty heavily on this linter, so it's important for us to have it actively maintained. IMO (and if they're up for it), @reedy and/or @umherirrender would be good candidates to help maintain these repos.

@umherirrender
Copy link

I cannot help on this project, sorry.

@grogy
Copy link
Member

grogy commented Aug 31, 2020

Is here somebody who can help and have a time? When no, I will this issue close.

When you can help, here are issues and PR are welcomed.

@jrfnl
Copy link
Collaborator

jrfnl commented Aug 31, 2020

I kind of have a feeling I have been helping quite a bit already....

@reedy
Copy link
Author

reedy commented Sep 1, 2020

When you can help, here are issues and PR are welcomed.

Sure they are welcomed, but it has taken quite a long time for numerous PR to be reviewed and merged.

Just on this task alone, it was opened on 13 May, no reply from the currenty maintainers until 30 August... That's 119 days; 3 months and 17 days

@grogy
Copy link
Member

grogy commented Sep 1, 2020

@jrfnl I agree, you make precision PR and it makes sense. You can be a maintainer when you want.

@reedy yes, it takes more weeks.

@villfa
Copy link

villfa commented Dec 1, 2021

Hi,
As a user of this tool I'm interested in seeing this conversation progress.

By reading this thread I have the feeling that @jrfnl is ok to be maintainer. Am I correct?

@grogy, were you waiting her explicit confirmation?

@jrfnl
Copy link
Collaborator

jrfnl commented Dec 1, 2021

By reading this thread I have the feeling that @jrfnl is ok to be maintainer. Am I correct?

Well, considering I'm doing a lot of the work already, it would make sense, but I would like to have a conversation with @grogy in that case about how to get to a workable collaboration. I generally don't merge my own PRs when I'm not the sole maintainer and the current time from PR to merge for most PRs, is, IMO, too long.

@villfa
Copy link

villfa commented Dec 1, 2021

@grogy If it can help please consider me as volunteer to be maintainer too.
I'm already the maintainer of churn-php. Take also a look at my profile and at my own repos. You'll see that I'm pretty active on Github.

@grogy
Copy link
Member

grogy commented Dec 1, 2021

Thanks for your words - to both. Tooooo long response to PR is my problem. I does not have so much time to maintaining :/

@villfa - please, try send some helpfull PR to this repo, after it we can continue in considering

@jrfnl - I trust you and to today i saw so many PR. I added you to maintainers. Thanks for your help :-))

@grogy
Copy link
Member

grogy commented Dec 1, 2021

@grogy grogy closed this as completed Dec 1, 2021
@jrfnl
Copy link
Collaborator

jrfnl commented Dec 1, 2021

@grogy Appreciated. Would you be open to that conversation I suggested in #32 (comment) ?

@glensc
Copy link

glensc commented Dec 1, 2021

🚧 Never link to branches, share a permalink 🚧

@grogy
Copy link
Member

grogy commented Dec 1, 2021

@jrfnl yes. My ideas are simple

  • I don't want the project to die
  • Focus on progress, not perfection
  • Makes me sense support old PHP versions
  • Backward compatibility
  • Simple code (for me simple code = less bugs, simple code = lower barrier for new contributor)

I am open for new look. Please, give me a feedback

@jrfnl
Copy link
Collaborator

jrfnl commented Dec 1, 2021

@grogy If you're open to it, let's talk this over in a call ? If you like, you can DM me on Twitter to set this up ?

My main concern at this moment is the response time for PRs.
I think it may also be a good idea to talk through things like merge protocols/conventions, release steps, possibly documenting some things in a CONTRIBUTING file.

The thing I'd like to improve most, for now, is maintainability and stability:

  • Things like, one class per file, fixing the namespace, updating the dependencies (Highlighter, Color) etc
  • Expand the testing so we can prevent (and find & fix) bugs instead ad-hoc fixing them when reported.
    Probably also a good idea to start monitoring code coverage.

I'm also thinking of things like making it compatible with the PHAR installer Phive to allow for wider usage .

I'm not sure what you mean by "Makes me sense support old PHP versions", but I'm used to working in projects with a wide PHP compatibility range, so I'm happy to continue to support the current range of version.
Might be a good idea to add PHPCompatibility to the PHPCS ruleset though.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Development

No branches or pull requests

7 participants