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

[Bug] scoop cat doesn't use bat #5217

Closed
milnak opened this issue Oct 21, 2022 · 0 comments · Fixed by #5247
Closed

[Bug] scoop cat doesn't use bat #5217

milnak opened this issue Oct 21, 2022 · 0 comments · Fixed by #5247
Labels

Comments

@milnak
Copy link

milnak commented Oct 21, 2022

Bug Report

Current Behavior

This is a follow up to bug #4905

Expected Behavior

It's expected that installing bat would use bat for scoop cat, but it doesn't. Instead, you need to explicitly set scoop config cat_style to enable this.

Additional context/output

This is confusing, as scoop help shows:

cat Show content of specified manifest. If available, ``bat`` will be used to pretty-print the JSON.

it should instead state that if configured, bat will be used, e.g. use scoop config cat_style plain, or perhaps it should use bat by default, if installed.

Possible Solution

@niheaven says:

We planned to enable bat by default, just as aria2 does. But using bat means more time to show the content, and change the default behavior. Maybe the info hasn't been changed.

System details

Windows 11, 64-bit, PS 7.2.6, Windows Terminal

Scoop Configuration

{
  "last_update": "2022-10-20T22:06:42.5802990-07:00",
  "cat_style": "plain",
  "alias": {
    "ua": "scoop-ua",
    "lsf": "scoop-lsf",
    "nfo": "scoop-nfo",
    "s": "scoop-s",
    "ls": "scoop-ls",
    "i": "scoop-i",
    "u": "scoop-u",
    "rm": "scoop-rm",
    "ca": "scoop-ca",
    "cu": "scoop-cu"
  },
  "aria2-warning-enabled": false,
  "show_manifest": true,
  "aria2-max-connection-per-server": "2",
  "scoop_repo": "https://github.com/ScoopInstaller/Scoop",
  "scoop_branch": "master"
}
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

Successfully merging a pull request may close this issue.

1 participant