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.

Sign up for GitHub

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

Jump to bottom

Use builtins instead of external commands #25

Open

a5ob7r wants to merge 3 commits into sheepla:main from a5ob7r:use_builtins

Contributor

a5ob7r commented Oct 31, 2021

Remove extra dependencies(cat, env, which), but maybe these are installed in almost all environments...

This makes a little bit fast this command invocation because no external command loads.

a5ob7r added 3 commits

October 30, 2021 01:19


          [refector] use builtin echo instead of cat

a29a260

Remove extra dependency. But maybe cat is installed in almost
environments...


          [refector] use builtin type instead of which

8c1cbe5


          [refactor] remove env

28e8595

It is redundant because Bash does the same thing at command invocation.

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