First off, congratulations for creating and maintaining this project!!
It's great and I really enjoy it!!
This pull request is simply to remove of two out-of-place forward slashes
( / ) in README.md, in the install instructions for Windows and Termux.
I know it's a small detail, but it was never fixed, and those forward slashes
cause the command to not execute properly, resulting in an useless
command that should delete a folder, but does nothing instead.
Pull Request Template
Type of change
[ ] Bug fix
[ ] Feature
[X] Documentation update
Description
In README.md - Install From Source,
for both Windows and Termux,
the last command 'rm -rf /ani-cli' doesn't work,
since there is no 'ani-cli' folder in the '/' directory.
It doesn't throw error because of the -f flag, but it
also fails to remove the intended folder. This commit
fixes this by removing the slash. Previous commands
use 'ani-cli' instead of './ani-cli', so removing the slash
seems to make more sense than adding a dot before it.
First off, congratulations for creating and maintaining this project!! It's great and I really enjoy it!!
This pull request is simply to remove of two out-of-place forward slashes ( / ) in README.md, in the install instructions for Windows and Termux.
I know it's a small detail, but it was never fixed, and those forward slashes cause the command to not execute properly, resulting in an useless command that should delete a folder, but does nothing instead.
Pull Request Template
Type of change
Description
In README.md - Install From Source, for both Windows and Termux, the last command 'rm -rf /ani-cli' doesn't work, since there is no 'ani-cli' folder in the '/' directory. It doesn't throw error because of the -f flag, but it also fails to remove the intended folder. This commit fixes this by removing the slash. Previous commands use 'ani-cli' instead of './ani-cli', so removing the slash seems to make more sense than adding a dot before it.