- Notifications
You must be signed in to change notification settings - Fork927
ci: standardize on go 1.22.12#17047
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 ourterms of service andprivacy statement. We’ll occasionally send you account related emails.
Already on GitHub?Sign in to your account
Uh oh!
There was an error while loading.Please reload this page.
Changes fromall commits
d98d3e8
fe48124
ab45772
d0c9cf8
File filter
Filter by extension
Conversations
Uh oh!
There was an error while loading.Please reload this page.
Jump to
Uh oh!
There was an error while loading.Please reload this page.
Diff view
Diff view
There are no files selected for viewing
Original file line number | Diff line number | Diff line change |
---|---|---|
@@ -2,14 +2,14 @@ FROM rust:slim@sha256:9abf10cc84dfad6ace1b0aae3951dc5200f467c593394288c11db1e17b | ||
# Install rust helper programs | ||
# ENV CARGO_NET_GIT_FETCH_WITH_CLI=true | ||
ENV CARGO_INSTALL_ROOT=/tmp/ | ||
RUN cargo install typos-cli watchexec-cli && \ | ||
There was a problem hiding this comment. Choose a reason for hiding this commentThe reason will be displayed to describe this comment to others.Learn more. review: | ||
# Reduce image size. | ||
rm -rf /usr/local/cargo/registry | ||
FROM ubuntu:jammy@sha256:0e5e4a57c2499249aafc3b40fcd541e9a456aab7296681a3994d631587203f97 AS go | ||
# Install Go manually, so that we can control the version | ||
ARG GO_VERSION=1.22.12 | ||
# Boring Go is needed to build FIPS-compliant binaries. | ||
RUN apt-get update && \ | ||
@@ -65,9 +65,6 @@ RUN apt-get update && \ | ||
# we're using for the version of go-critic that it embeds, then check | ||
# the version of ruleguard in go-critic for that tag. | ||
go install github.com/quasilyte/go-ruleguard/cmd/ruleguard@v0.3.13 && \ | ||
Comment on lines -68 to -70 There was a problem hiding this comment. Choose a reason for hiding this commentThe reason will be displayed to describe this comment to others.Learn more. review: this was failing to build with a tools dependency issue but it doesn't seem to be required since go 1.18? | ||
# go-releaser for building 'fat binaries' that work cross-platform | ||
go install github.com/goreleaser/goreleaser@v1.6.1 && \ | ||
go install mvdan.cc/sh/v3/cmd/shfmt@v3.7.0 && \ | ||
@@ -128,6 +125,7 @@ RUN apt-get update --quiet && apt-get install --yes \ | ||
asciinema \ | ||
bash \ | ||
bash-completion \ | ||
bat \ | ||
bats \ | ||
bind9-dnsutils \ | ||
build-essential \ | ||
@@ -140,6 +138,7 @@ RUN apt-get update --quiet && apt-get install --yes \ | ||
docker-ce \ | ||
docker-ce-cli \ | ||
docker-compose-plugin \ | ||
exa \ | ||
fd-find \ | ||
file \ | ||
fish \ | ||
@@ -176,6 +175,7 @@ RUN apt-get update --quiet && apt-get install --yes \ | ||
postgresql-16 \ | ||
python3 \ | ||
python3-pip \ | ||
ripgrep \ | ||
rsync \ | ||
screen \ | ||
shellcheck \ | ||
Uh oh!
There was an error while loading.Please reload this page.