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

Is this project dead? #243

Closed
gilles-crealp opened this issue Jun 7, 2022 · 10 comments
Closed

Is this project dead? #243

gilles-crealp opened this issue Jun 7, 2022 · 10 comments
Assignees

Comments

@gilles-crealp
Copy link

No answers from @mattallty since 2020...

@jhorbulyk
Copy link
Contributor

My remarks:

  • I don't know if anyone else has write/merge permissions for this repo.
  • Most of the open PRs are either bots updating dependencies or minor typos. There costs of not merging those PRs aren't particularly high.
  • I personally wouldn't be opposed to being a maintainer of a living fork of this project if there was a need for changes to go ahead.

@ryanrubleycoates
Copy link

Watching with interest, as we use this in a few projects and the warnings are becoming annoying. Would love to move over to a fork if somebody is going to pick it up

@cah4a
Copy link

cah4a commented Jun 18, 2022

Caporal.js is now the best solution I've seen. Would love to use a maintained fork.

@donmccurdy
Copy link
Contributor

I'm using Caporal in my own project (github.com/donmccurdy/glTF-Transform/) and have gotten complaints from downstream users about the npm audit warnings from old dependencies. I have a pretty dim view of npm audit in general, but it would still be nice to keep these dependencies up to date, even if new feature development isn't planned.

After replacing GitHub's dependabot with Renovate (https://github.com/renovatebot/renovate) (free for open source) I've found that quite helpful. It can batch dependency updates into a single PR on a fixed schedule (weekly, monthly, ...) and merge the PR automatically if tests pass. I'd be willing to help set up something like that here if maintainers are interested.

@ryanrubleycoates
Copy link

We gave up and migrated everything to Commander with minimal pain today, and realized that it shaved about 3MB from our compressed .deb distributions as a bonus. I guess there are a LOT of deps in Caporal for things we didn't even use.

@donmccurdy
Copy link
Contributor

donmccurdy commented Aug 21, 2023

I've gone ahead with forking the project, updating dependencies, and publishing a new build on npm as @donmccurdy/caporal. Feel free to use my fork if you'd like:

https://github.com/donmccurdy/Caporal.js

Unless the situation changes on the upstream project, I'll plan to strip down the build system to something I'm more comfortable maintaining, add Renovate for automatic dependency updates, and leave things alone beyond that. I don't have any personal plans to add features or change the API — Caporal has worked really well for me, and migrating to other CLI generators didn't make sense in my project. If others are interested in getting involved within or beyond that scope, feel free to comment over there.

@mattallty
Copy link
Owner

mattallty commented Aug 26, 2023

Hey - mostly dead as you can see :(
The good news is I can work on it again more easily since I just lost my job, but the best advice I could give for now is to use the fork of @donmccurdy: https://github.com/donmccurdy/Caporal.js

My plans are:

  • clean the repo of vulnerabilities
  • work on a v3, but be aware that it will be full of breaking changes since I've a totally different interface in mind.

@mattallty
Copy link
Owner

@gilles-crealp @cah4a @donmccurdy

I've spent the night re-looking at it. Not really sure it needs a V3 now.
I've a PR here that fixes a lot but I'd be happy if some folks could test on some real use cases:

#253

@mattallty
Copy link
Owner

@gilles-crealp @cah4a @donmccurdy

You should be able to test it using npm install @caporal/core@next
i've been able to successfully test it on my side

@mattallty
Copy link
Owner

3.0 has been released (major update because it requires Node 16+)
Closing this issue

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

No branches or pull requests

6 participants