Skip to content

Added farjs-app to command-line-utilities.json #53

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

Closed
wants to merge 1 commit into from
Closed

Added farjs-app to command-line-utilities.json #53

wants to merge 1 commit into from

Conversation

viktor-podzigun
Copy link

Thank you for contributing to the Node.js Toolbox catalog! 🙏

Note: Node.js Toolbox is specifically for backend libraries. If a library works both in the browser AND in Node.js feel free to add it. Libraries that work ONLY in the browser are out of scope for now.

Before submitting your pull request, please make sure:

  • To use the exact package name as it appears on NPM (and npm install <package-name>)
  • The packages are listed in alphabetical order
  • The package doesn't already exist in a different category
  • Description doesn't have a dot (.) at the end
  • To format with Prettier before committing (npm run prettier)

@viktor-podzigun
Copy link
Author

Was not sure about the right category. Seems like it is the closest, though it is the cli app, not a lib, maybe it is time for a new category? 🤔

@Maximization
Copy link
Owner

@viktor-podzigun Thank you for your contribution. Had a look at farjs-app and correct me if I'm wrong but it's not really a library you import/require in Node.js and use it in a project. Nor is it built with Node.js. For these reasons, I don't think it belongs in Node.js Toolbox.

@viktor-podzigun viktor-podzigun deleted the patch-2 branch August 12, 2024 12:36
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

Successfully merging this pull request may close these issues.

2 participants