-
Notifications
You must be signed in to change notification settings - Fork 424
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
Is this project no longer being actively maintained? (Sept 2015) #399
Comments
I'm all for that. Honestly this project should probably be deprecated. There are a lot of improvements to be made, but it is becoming a big pile of spaghetti code, and is very difficult to maintain. Almost every patch introduces new bugs for someone else. |
As a collaborator, do you have any recommendations for an alternative? |
@Zamiell you could fork it and merge in all the prs you want. There are no alternative afaict. But I'd look at https://github.com/oftn/oftn-bot and try to extract what you need |
Here are two that I've played with. https://github.com/gf3/IRC-js |
First one hasn't been updated in a year, too. If PRs start getting merged in again, might be worth focusing on code cleanup for this lib. Still seems to be the best option out there |
@IanMitchell but like, no one steps up so ... |
Yeah, but that might be because PRs are inactive - a lot are outstanding with no comments on them. Why put the effort in if it'll just sit in a branch forever? |
With a marked name like 'irc' in node: "npm install irc", i think its worthwhile to revive, this will always generate lots of traffic. Is it possible to rollback to a good collaborative revision and re-implement? I will help where i can, the code looks workable, i would suggest going through the oldest pull requests, and then seeing if those contributors are still responsive so we can get their code in.. |
the |
It sounds like a good problem to have with so many contributions. I just hope I'm not to late to the party, this seems to be the most user friendly project of its kind, even out of the ones just previously mentioned. I suggest the name be kept as long as possible though, I'm sure a lot of people still depend on it. |
I'll start heading through the PRs and merge what I can. A big issue we've had is people submitting PRs, and never coming back to respond to comments. It is hard for me to make the time fix up every PR that comes in. I'll make a goal to have the backlog as clean as I can get it, and publish a new release this coming Monday(9/7). |
Jirwin, anything i can do? Funny, thats my birthday. |
Fantastic jirwin, thanks so much. |
@jnull It'd be easier to coordinate if you joined ##node-irc on freenode so we could chat. |
I would close all the PRs that are abandoned. |
It seems to have been maintained a little in the meantime, but it still has a few major issues and a lot of unmerged pull requests (possibly broken?). I've started a fork at https://github.com/Throne3d/node-irc which has already fixed a few major issues, without (I think) introducing new bugs. |
With 52 open issues, 32 open pull requests, and the last code change being 4+ months ago (other than a commit fixing a typo in the README.md file)...
Is it time to start talking about having some new people take the reins?
The text was updated successfully, but these errors were encountered: