Skip to content
Open
Changes from all commits
Commits
File filter

Filter by extension

Filter by extension

Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
4 changes: 3 additions & 1 deletion README.md
Original file line number Diff line number Diff line change
Expand Up @@ -29,7 +29,9 @@ brew install openssl readline
### Debian/Ubuntu

```bash
sudo apt-get install build-essential libreadline-dev zlib1g-dev libssl-dev uuid-dev
sudo apt-get install -y \
build-essential libreadline-dev zlib1g-dev libssl-dev uuid-dev \
bison flex perl pkg-config libicu-dev
Comment on lines +32 to +34
Copy link
Copy Markdown

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

medium

For consistency with the plugin's internal error messages (see hooks/post_install.lua line 211) and modern Debian/Ubuntu best practices for interactive documentation, consider using apt instead of apt-get. Additionally, while the -y flag is convenient for automation, it is often omitted in documentation to allow users to review the packages being installed before the process begins.

Suggested change
sudo apt-get install -y \
build-essential libreadline-dev zlib1g-dev libssl-dev uuid-dev \
bison flex perl pkg-config libicu-dev
sudo apt install -y \
build-essential libreadline-dev zlib1g-dev libssl-dev uuid-dev \
bison flex perl pkg-config libicu-dev

```

### RHEL/CentOS
Expand Down
Loading