Skip to content

tldr --list output should be not in a list #230

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
fazlearefin opened this issue Feb 22, 2024 · 3 comments
Closed

tldr --list output should be not in a list #230

fazlearefin opened this issue Feb 22, 2024 · 3 comments

Comments

@fazlearefin
Copy link
Member

Running tldr --list outputs a long single line containing a list. This makes it difficult to grep for a particular command of interest (unless grep complex regex is used). tldr --list should output available commands separated by newlines just like the official rust client.

tldr --list | grep docker
# OH MY!
@vitorhcl
Copy link
Member

Is your version updated? This is a duplicate of #221 fixed by #222, actually.

@fazlearefin
Copy link
Member Author

fazlearefin commented Feb 24, 2024

Yes, version is updated. Looks like no new release was made after the fix.

Please see screendump below:

root@4f9f90526bf9:~#
root@4f9f90526bf9:~# pip3 install tldr
Collecting tldr
  Downloading tldr-3.2.0-py3-none-any.whl (11 kB)
Collecting termcolor
  Downloading termcolor-2.4.0-py3-none-any.whl (7.7 kB)
Collecting colorama
  Downloading colorama-0.4.6-py2.py3-none-any.whl (25 kB)
Collecting shtab>=1.3.10
  Downloading shtab-1.7.0-py3-none-any.whl (14 kB)
Installing collected packages: termcolor, shtab, colorama, tldr
Successfully installed colorama-0.4.6 shtab-1.7.0 termcolor-2.4.0 tldr-3.2.0
WARNING: Running pip as the 'root' user can result in broken permissions and conflicting behaviour with the system package manager. It is recommended to use a virtual environment instead: https://pip.pypa.io/warnings/venv
root@4f9f90526bf9:~#
root@4f9f90526bf9:~# tldr --version
tldr 3.2.0 (Client Specification 1.5)
root@4f9f90526bf9:~#
root@4f9f90526bf9:~# tldr --list
[]
root@4f9f90526bf9:~# tldr --update
Updated cache for 4813 entries
root@4f9f90526bf9:~#
root@4f9f90526bf9:~# tldr --list
['apx-subsystems', 'acountry', 'e2image', 'steghide', 'blkid', 'prename', 'sed', 'pmap', 'sport', 'btrfs-scrub', 'rtorrent', 'pdfcrop', 'paru', 'systemd-ac-power', 'gsettings', 'rc-update', 'qm-disk-resize', 'reflector', 'blastn', 'chkconfig', 'mac2unix', 'drawing', 'cpufreq-set', 'v4l2-ctl', 'gedit', 'pacman-mirrors', 'mkswap', 'debootstrap', 'lvs', 'compress', 'lrzip', 'reboot', 'speedometer', 'cam', 'fold', 'debugfs', 'toolbox-enter', 'getopt', 'resize2fs', 'nordvpn', 'cgroups', 'getfacl', 'swaplabel', 'qm-showcmd', <snip>

@kbdharun
Copy link
Member

kbdharun commented Feb 24, 2024

Yes, version is updated. Looks like no new release was made after the fix.

Yep, I think we can do a release here post implementing the placeholder escaping part (btw I am planning to make a Client specification release this weekend in the main repo, so the Python client caching source would need to be updated too).

For now, I will mark this issue as completed.

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

3 participants