Skip to content

feat: remove awscli and jq as a dependency; remove erl crash dump#1837

Open
filipecabaco wants to merge 1 commit intomainfrom
feat/reduce-runtime-dependencies
Open

feat: remove awscli and jq as a dependency; remove erl crash dump#1837
filipecabaco wants to merge 1 commit intomainfrom
feat/reduce-runtime-dependencies

Conversation

@filipecabaco
Copy link
Copy Markdown
Member

What kind of change does this PR introduce?

remove awscli and jq as a dependency; remove erl crash dump

@coveralls
Copy link
Copy Markdown

Coverage Status

coverage: 91.912%. remained the same — feat/reduce-runtime-dependencies into main

Comment thread Dockerfile

RUN apt-get update -y && \
apt-get install -y libstdc++6 openssl libncurses5 locales iptables sudo tini curl awscli jq && \
apt-get install -y libstdc++6 openssl libncurses5 locales iptables sudo tini curl && \
Copy link
Copy Markdown
Contributor

@leandrocp leandrocp Apr 29, 2026

Choose a reason for hiding this comment

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

The only small suggestion would be removing sudo because the container is already running as root but it saves only a few kb so it doesn't matter much

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

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants