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

FEATURE - Consider moving /bin/* to another repo #1634

Open
robertsLando opened this issue Jul 11, 2023 · 3 comments
Open

FEATURE - Consider moving /bin/* to another repo #1634

robertsLando opened this issue Jul 11, 2023 · 3 comments

Comments

@robertsLando
Copy link
Member

Many deps are related to bin/*:

  • commists
  • concat-stream
  • help-me
  • minimist
  • split2

So this are 5 deps on a total of 15. I would suggest to create a new dedicated repo @mqttjs/cli and move that code there.

@vishnureddy17
Copy link
Member

I wonder how valuable the CLI really is. Since we are making breaking changes anyway, maybe we should just consider removing it altogether.

@robertsLando
Copy link
Member Author

robertsLando commented Jul 11, 2023

I think the CLI is cool even if I really dunno how many users are using it right now.

I don't think creating a new repo adds so much effort, I would simply copy paste the code as it is and publish it to npm. I also saw there is actually no test that covers that part

Anyway there is also https://github.com/hivemq/mqtt-cli that does the same (I think better)

Copy link

This is an automated message to let you know that this issue has
gone 365 days without any activity. In order to ensure that we work
on issues that still matter, this issue will be closed in 14 days.

If this issue is still important, you can simply comment with a
"bump" to keep it open.

Thank you for your contribution.

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

No branches or pull requests

2 participants