-
Notifications
You must be signed in to change notification settings - Fork 35
new release #43
Comments
Any plans or updates concerning a release? The library hasnt been published in 2 years and there have been significant changes since then (specifically the removal of the lodash dependency). |
@jamestalmage Please? People are waiting for this for a while now :'( |
i made handy release for this issue. https://www.npmjs.com/package/@taichi/cli-table2 |
@jamestalmage let us know if and we can help you here or if you need some more maintainers =) |
Agreed, we are also running into this on a few repos, for example: More than happy to help out with getting a release out there. Perhaps we can integrate semantic-release[https://github.com/semantic-release/semantic-release] to aid here? Happy to open a PR to add that. Let me know @jamestalmage |
maybe we should fork this into |
@Turbo87 that is exactly what I was thinking. I was thinking of doing it myself but, there are unanswered questions that needs answering. How much maintenance will be required for whomever forks it? Are there people that know enough about this lib that can support it going forward? There are currently 23 issues and 3 open pull requests. It needs a resolution though, we just need to be sure that once it is forked, it will be supported by more than one person. |
@schalkneethling I think the important part would be to not have it on the GitHub account of a single developer but create a GitHub org instead. In this issue I see at least 3-4 potential maintainers and I think if we share the load this should be manageable. I'm happy to setup the GH org, repo and CI if we agree that this is the best way forward. |
the other alternative would be to reactivate |
An org is definitely a good idea and would increase the truck factor. |
@Turbo87 @DanielRuf That is a great idea. I will definitely support the move to an org and getting a couple of maintainers to share the load. 👍 |
https://github.com/cli-table/cli-table3 you should get invites to the org in the next minutes :) |
published the current |
@Turbo87 You mean |
whoops, yeah, that's what I meant to write |
@Turbo87 I guess we need to notify or open pull requests against packages that depends on cli-table2 to inform them of the new canonical repo? Thanks for getting this all set-up o/\o |
yeah, that would a good idea. I've started by opening a PR at npm/npm#20955. do you know other projects that use |
@Turbo87 Good question, let me see what some of my Snyk reports say. Also, we should probably set-up some kind off communication channel for the project. Perhaps Gitter or Slack? |
@Turbo87 I guess the list of dependants here would be projects that depend on That is a lot of projects. |
455, we should create a list then =) https://www.npmjs.com/package/cli-table2?activeTab=dependents |
I'll try to help creating PRs as much as possible. |
Yup, and we should probably carry over the open issues from here to the new repo. Those that are still relevant. Also, there are 3 pull requests. If they are still relevant, we should probably ask the authors to open them against the new repo. Thoughts? |
@DanielRuf @Turbo87 ok, there you go cli-table/cli-table3#17 :) |
Probably good as list with checkboxes to see the status in the overview (todo list, x / y done). |
Let me know if I should help here (transforming the list). |
@DanielRuf @Turbo87 should we migrate the issues over from here to the new repo? |
I'm not sure, some of them might not be relevant anymore... |
@Turbo87 very true, I was thinking if we want to triage it in some way? Perhaps look over them when we have time and add a comment with a 👍 or 👎 |
This PR updates the cli-table2 dependency to cli-table3, which fixes one of the npm audit warnings :) cli-table2 (like cli-table itself) is no longer maintained. In jamestalmage/cli-table2#43 a couple of people have offered to take over maintenance but the current maintainer did not respond so as a result the project was forked to cli-table/cli-table3.
Please make a new release and push to npmjs.com.
The text was updated successfully, but these errors were encountered: