Skip to content
This repository has been archived by the owner on May 14, 2024. It is now read-only.

About the state of maintenance #570

Closed
jsumners opened this issue Sep 29, 2019 · 4 comments
Closed

About the state of maintenance #570

jsumners opened this issue Sep 29, 2019 · 4 comments
Milestone

Comments

@jsumners
Copy link
Member

This issue is to give clear notice about the current state of maintenance of this module.

In July of 2019 I offered to pick up the maintenance of this project from Joyent. Subsequently, the repo was transferred to the ldapjs org and I was granted this role. When I offered to take over I was upfront about not being able to dedicate much more time than was already being dedicated to this module, but I would be able to dedicate at least something. That has not changed, but that promise is in an obscure place so I am bring it forth here directly.

My time for this project is very limited. The only time I have to work on it is on weekends or vacation days. And then only when I don't have other things going on and have the will to work. As you can probably imagine, this will make for slow going on the project.

My plan for a next release:

  1. "Clean up" the code base as much as possible without changing the outward functionality. By "clean up" I mean move shared code into independent testable modules (see the new modules being developed in the organization) and generally making it easier for people to get into the code and contribute to it.

  2. Resolve open issues. I don't want to outright close issues for no reason, but if the author has abandoned the issue by not responding when prompted, that is likely to happen.

  3. Resolve open PRs. I really don't want to just close these without due consideration.

There's a lot of work in all three of those things. If you wait on me and @UziTech to do it all ourselves then your wait will continue to be a very long one. If you have an issue, then dig into the code, figure out where it lies, and submit a PR that either 1) outright fixes it or 2) introduces a failing test in the test suite that covers it. Do not expect issues without accompanying PRs to get resolved quickly.

You can also help by working on current issues or resolving current PRs. In the case of a current PR, it would be ideal if you can branch off of that person's branch so they get credit for their work. But since all PRs must target the next branch before they will be considered for merging, that may not be feasible. Do your best.

After the next release, whenever that happens, I plan to expand the list of people who can directly commit to this repository. Until that time, I want to be able to coordinate efforts without conflicts.

@jsumners jsumners added this to the 2.0.0 milestone Sep 29, 2019
@jsumners jsumners pinned this issue Sep 29, 2019
@DavidIHunt
Copy link

If you do nothing else on this project, please PLEASE merge this...
#433

It's been 2 years!

The bug it fixes is making this module awful to use. Imagine trying to call a function of a library which can asynchronously throw an error that you can't catch and which will crash your process.

@hongkongkiwi
Copy link

Thank you thank you thank you!

How can we help? I'm a bit overwhelmed and confused with all the pull requests here and things. Is there a way we can contribute ?

@jsumners
Copy link
Member Author

@hongkongkiwi just start with one PR or issue that attracts you. This is exploratory for all of us.

@jsumners
Copy link
Member Author

All of my statements in this issue still apply. But v2.0.0 has been released, so I am closing and locking this.

@ldapjs ldapjs locked as resolved and limited conversation to collaborators May 31, 2020
@UziTech UziTech unpinned this issue Aug 30, 2021
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

No branches or pull requests

3 participants