-
Notifications
You must be signed in to change notification settings - Fork 317
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
Add some collaborator to maintain this package #334
Comments
Hi, @pahen @PabloLION I became interested in this project through developing #341 and discovered this issue while researching. I am not sure if I am qualified to be a maintainer for a project of this size, but I may be able to help since I develop some npm library. I can't commit to helping forever either, but how about trying to help out by focusing on maintaining? Since I can't see the status of my work and personal activities yet, I will try to volunteer until the end of March 2023 to work on fixing high priority bugs like Node.js 18 support. (It would be beneficial for me to contribute to this project, especially since getting #341 merged will increase the use of my project.) Thanks. |
Hi and sorry for not responding! I've now added you both as collaborators to this repo @kamiazya @PabloLION |
@pahen Thank you very much. I will try to focus on maintenance for a while, as per the above policy. |
Thank you! 🥇 |
@pahen I want to merge and release the first pull request you approve, is that okay? I assume CircleCI is supposed to automatically release the package.json when the version changes, but please let me know if there is another way 🙏 (Edit) |
The release step is done manually today. I'll add a NPM task to create a release and publish it to NPM to make it easier. PR is coming. |
I've added docs about creating releases now in https://github.com/pahen/madge#creating-a-release @kamiazya @PabloLION |
Thanks @pahen ! Will work on it later. |
@PabloLION DM is now open! |
@pahen @PabloLION I will merge #342, but the release of 4.0.0, which previously raised the minimum version of Node.js, was done with a major version upgrade so I will follow that policy and release the next version at 6.0.0. |
@pahen @PabloLION I have created a milestone to align our perceptions about the release target for v6.0.0. Please let us know if you have any concerns! |
Once #321 is merged, it will be ready for release. Requested @PabloLION to review #321. @pahen @PabloLION |
Great job @kamiazya thanks for pushing so actively. I was also added as collaborator. Because of my newborn and new projects I will not have plenty of time, but I am willing to support this project, too 👍 |
Nice works @kamiazya . And GitHub Discussions seems good. Could you please activate it @pahen? |
I'm now trying to label all the issues, and I feel this one can be closed already. |
I feel that there are still some issues where the project is still in the genus. I want to keep this open for a while 🙏 |
My day job depends on madge and I have some experience with maintenance. I think it's great of @kamiazya and @PabloLION to take the lead here - just letting you know that you may @ me here if I may be of assistance to unblock anything for now. |
As discussed in #319 (comment), this repo is currently down for maintenance and we have 0 issue solved / PR merged in the last month (See here https://github.com/pahen/madge/pulse/monthly)
I believe that this repo needs some helping hands from the community. As I have no experience maintaining any popular package, maybe some more senior ones can help with this?
The text was updated successfully, but these errors were encountered: