CI "minimum" leg builds with go1.26.11, so the go 1.26.0 floor is untested
Mantenedores costumam responder em até 1 dia
Ninguém assumiu esta issue ainda.
Avaliação
- Dificuldade
- 2/5
- Tempo estimado
- 1-3 horas
- Facilidade para iniciantes
- 62/100
Direção de pesquisa
Comece em .github/workflows/go.yml, onde o leg build (minimum) usa actions/setup-go com go-version-file: 'go.mod', o que instala a linha toolchain (go1.26.11) em vez da diretiva go 1.26.0 em go.mod. Você pode passar a versão da diretiva go para o setup-go como go-version (lida com uma etapa sed) ou reescrever o comentário e o nome do leg para dizer que ele testa a linha de toolchain. Está concluído quando o log do leg mostra a versão da diretiva, ou quando o comentário corresponde ao que é testado, e a decisão do piso 1.26.0 está anotada ao lado da regra de ignorar lipgloss em .github/dependabot.yml.
Escrita pelo modelo de indexação a partir do texto da issue.
Descrição
Summary
The CI leg that claims to prove the go.mod floor installs the toolchain version instead, so nothing builds or tests with Go 1.26.0, the version that also holds back a dependency upgrade.
Where
.github/workflows/go.yml:11.github/workflows/go.yml:28go.mod:3go.mod:5.github/dependabot.yml:15
Details
go.yml:11-13 says "minimum" proves the go.mod floor (1.26.0) still builds and passes, and the leg uses actions/setup-go with go-version-file: 'go.mod' (go.yml:28-33). go.mod has go 1.26.0 (line 3) and toolchain go1.26.11 (line 5); setup-go takes the toolchain line. The log of job build (minimum) in run 43648865601 (head d0c56f0) reads Setup go version spec 1.26.11, Successfully set up Go version 1.26.11, go version go1.26.11 linux/amd64, with GOTOOLCHAIN: local. No other job installs 1.26.0 (the rest use stable, which was go1.27.4, or the same go.mod file).
Impact
No CI job has built or tested the code with Go 1.26.0 since the toolchain directive was added (#243). The floor is what keeps lipgloss on v2.0.x (dependabot.yml:15-19 ignores >= 2.1.0 because it needs go 1.26.7; docs/DEVELOPER_GUIDE.md:340-341 repeats the rule), so an available upgrade is held back for a minimum that nothing verifies. A user who builds with GOTOOLCHAIN=local on 1.26.0 to 1.26.10 is on an untested path. The "Check go directive" step only compares the text of the line.
Steps to reproduce
gh api repos/TMHSDigital/subenum/actions/jobs/132612623946/logs, then search for Setup go version spec.
Suggested fix
Make the leg install the go line's version: read it in a step (sed -n 's/^go //p' go.mod) and pass it to setup-go as go-version (setup-go already sets GOTOOLCHAIN=local, so 1.26.0 is then what compiles). Or, if 1.26.0 is not really meant to be supported, say in the comment that the leg tests the toolchain line, raise the go directive on purpose, and drop the lipgloss ignore.
Notes
Related: #243 (added the toolchain directive).
Acceptance criteria
- The leg's log shows the version of the
godirective, or the comment and leg name say it tests the toolchain line - The decision about the 1.26.0 floor is written down next to the Dependabot ignore rule
Getting started
- Relevant files:
.github/workflows/go.yml,go.mod,.github/dependabot.yml - How to run tests:
make test(orgo test ./...). Tests must stay hermetic: use-simulate,-simulate-zoneor the local DNS test server, never a live domain. - Related docs:
docs/DEVELOPER_GUIDE.md,docs/ARCHITECTURE.md
Filed by repo-audit/full-audit at d0c56f0. Severity: low. Effort: S. audit-key: ci|.github/workflows/go.yml|build.minimum.go-version-file
- Linguagem predominante
- Go
- Estrelas
- 1
- Forks
- 1
- Merge médio
- 6h 42min
- PRs com merge (30d)
- 7
Preparar o ambiente
Inicia o contêiner de desenvolvimento do projeto no navegador, com a sua própria conta do GitHub.
- Inclui um Dockerfile ou arquivo Docker Compose
- Tem um modelo de pull request
- Ler o guia de contribuição
Primeiros passos
- Leia a issue inteira e depois o guia de contribuição do projeto.
- Comente na issue dizendo que vai assumir — evita que duas pessoas façam o mesmo trabalho.
- Faça um fork do repositório e trabalhe em uma branch.
- Abra um pull request que referencie o número da issue.
Mais de TMHSDigital/subenum
-
good first issue severity: low tech-debt
Dificuldade 2/5 Meio dia Facilidade para iniciantes 75/100
TMHSDigital/subenum#329 ·
Mantenedores costumam responder em até 1 dia
-
area: docs documentation good first issue severity: low
Dificuldade 2/5 1-3 horas Facilidade para iniciantes 85/100
TMHSDigital/subenum#327 ·
Mantenedores costumam responder em até 1 dia
-
area: release dependencies good first issue severity: low
Dificuldade 2/5 1-3 horas Facilidade para iniciantes 66/100
TMHSDigital/subenum#324 ·
Mantenedores costumam responder em até 1 dia
-
area: output bug good first issue severity: low
Dificuldade 2/5 1-3 horas Facilidade para iniciantes 76/100
TMHSDigital/subenum#319 ·
Mantenedores costumam responder em até 1 dia
-
community documentation good first issue
Dificuldade 2/5 1-3 horas Facilidade para iniciantes 85/100
TMHSDigital/subenum#135 ·
Mantenedores costumam responder em até 1 dia
Todas as issues de TMHSDigital/subenum
Issues semelhantes
-
Dificuldade 2/5 1-3 horas Facilidade para iniciantes 62/100
GillMacedo/godebugdiff#13 ·
-
Dificuldade 2/5 1-3 horas Facilidade para iniciantes 78/100
GillMacedo/racesift#14 ·
-
IsTextFile reads the whole file into memory (OOM on large files in the file watcher)Talvez já em andamento Um pull request vinculado a esta issue está aberto ou já foi mesclado. Aberta
Dificuldade 2/5 1-3 horas Facilidade para iniciantes 68/100
gtsteffaniak/filebrowser#3395 ·
Mantenedores costumam responder em até 1 dia
-
effort:s impact:low raised-by:worker risk:low
Dificuldade 1/5 Menos de uma hora Facilidade para iniciantes 70/100
medici-finance/assay#6496 · 1 comentário ·
Mantenedores costumam responder em até 1 dia
-
kind/bug require/reproduce require/severity
Dificuldade 2/5 1-3 horas Facilidade para iniciantes 75/100
Mantenedores costumam responder em até 1 dia