-
Notifications
You must be signed in to change notification settings - Fork 186
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
Call for maintainers #192
Comments
@rosston we may get more attention if we add this to the Readme.me I think |
Currently working to add a link to here from the README. |
I'm interested in helping maintain this package. What is your process for making decisions on that front? |
@crellison I don't think we have a process for making decisions, that's one thing we should start working on. However, one requirement I think it's important is being using this package. Are you using the package? |
@bithavoc Sorry for the delayed response. I am using the package, though currently on a closed-source project. I can put together something you can look at if you want to validate that I know what I'm doing. |
sorry I was off for a few days. If you're using express-winston that's the only thing that matters. I'm gonna make you contributor so you can start addressing pull requests and issues in the repo. |
We need more maintainers. |
@crellison (and any other possible maintainers that crop up) I should've noted this in the first place, but I'm happy to help by providing backup, a second eye, or any background around express-winston. I'm not able to put a ton of time towards this module, but I can absolutely provide support to other maintainers. I worked on a lot of the structure of the test suite, so especially feel free to reach out with any questions about confusing things there. |
That all sounds good. What's the process for publishing to NPM? I'm looking to close #190, but want to make sure the changes get published so we can close some of the winston 3.x compatibility issues |
@crellison I can assist publishing to NPM, it takes me less than a minute to do so. |
Y'all are welcome to decide differently, but I've tried to keep the changelog updated (manually) when releasing to npm in the past. |
@rosston is right, we need to keep it updated |
@bithavoc I forgot to mention that the Changelog has been updated. Feel free to release 3.0.1 when you can! |
I am interested in contributing. I am using the package for work. |
@nagapavan good, you can start by going through the pull requests and vetting them to see what we should be merging or closing. you can summon @crellison if something needs more reviewing. |
:lurkyturkey: 👋 I haven't contributed in a really long time, and I don't think I'm going to be able to any time soon either unfortunately, but this recent event is something to keep in mind as you work on finding someone else to own this. ❤️ |
@jlomas-stripe Thanks for the reminder, yes I saw that one in HN. Unlike event-stream I still retain NPM publishing rights and check what we publish. |
I'm somehow new on github. I'm really intrested in this project and find it so cool. I'm available to be an active maintainer on this project. Here is a my framework (as an example). |
Can help contribute and being an active maintainer. ✋ |
Needs to be a link from the main winston readme, to this package. A section called "ecosystem" or something like that. I'm sure many people don't even know about this lib. If there were more users, maybe there'd be more interest in maintenance. |
I would like to help maintain this package. I like it a lot. I am using it for an open-source project currently and will probably make it something I use in all express-based projects. I can be on weekends and some weekdays. |
I made a pretty large pull request #222 . Is anyone still maintaining this library? Did anyone take over? |
@yinzara Unfortunately we still don't have active maintainers for the library, the most active maintainer right now is @golopot and me, but as soon as a I saw that the PR had a breaking change then I deferred for review when I had some more free time(like today Saturday, a weekend). After your PR gets merged, and if you volunteer, we can definitely promote you. Sounds good? I'm still reviewing all the PR's and Releasing the versions to NPM for now. |
I mean, what's another library to maintain? :-) Why not!? |
@yinzara releasing two new versions right now, sorry it took me so long, I had a busy weekend. |
I think I'm still technically a contributor as I can't close any issues. Would love to get rid of all the old stuff that isn't valid anymore. |
What do I need to do to become a contributor? Anything special? Or can I just start looking through PRs? |
@jblevins1991 basically just look through PR and watch the repo for new issues and PRs, yes. |
🚧 Is this repo looking for support? |
@bithavoc I have been using this repo for a while. I had some free time and wanted to update a few things since i saw this #192 call for maintainers i would like to offer to do some repo maintenance. If you don't mind I am going to create a few issues and solve a few existing. |
@bithavoc nudging you a little bit again this year :D I am still available to make contributions, also an active user of this repo and would love to tidy it up a bit. |
Is this project still maintained? If not, is there a fork or alternative that people recommend? |
This project is fairly popular (40,000-50,000 downloads per week on npm), but it doesn't really have an active maintainer at the moment. I was (more or less) keeping up with issues and pull requests for a while, but my ability/interest in contributing has been low for the past year or so.
If anyone is interested in maintaining this project, please speak up! Responsibilities would basically be triaging/responding to/closing issues and handling incoming PRs. From a skills standpoint, you would benefit from some knowledge of JavaScript (don't think you have to be an expert!), patience in handling issues and PRs, and an understanding of the balancing act between new features and stability.
The project is not in a dire situation at the moment. It generally works as advertised, although there might be some clunkiness around the current winston@3 support (see #189 and #190). The biggest debt on the project is the number of unhandled issues and PRs open, which doesn't actively hurt the currently-deployed code. But the project could use some love, which I'm not able to give it.
The text was updated successfully, but these errors were encountered: