CI: cross-compile and vet for Windows
Nadie ha tomado este issue todavía.
Evaluación
- Dificultad
- 2/5
- Tiempo estimado
- 1-3 horas
- Aptitud para principiantes
- 88/100
Línea de trabajo
Comienza en .github/workflows/ci.yml, en el job build-test, y revisa los targets existentes de .goreleaser.yaml. Añade compilación cruzada para Windows amd64 y arm64, además de GOOS=windows go vet ./..., y luego verifica que el workflow detecte código específico de Windows como internal/deps/exec_windows.go y deps_test.go sin ejecutar pruebas de Windows.
Escrito por el modelo de indexación a partir del texto del issue.
Descripción
.goreleaser.yaml ships windows/amd64 and windows/arm64, but CI (.github/workflows/ci.yml, build-test) only builds and vets on ubuntu-latest for the host platform. #52 splits the process-kill code into internal/deps/exec_unix.go (!windows) and exec_windows.go (windows), so code that only compiles on Windows is now in the tree. If it breaks, CI won't catch it; the first sign would be a failed release.
Proposal
Add cross-compile steps to build-test:
- name: Cross-compile (windows)
run: |
GOOS=windows GOARCH=amd64 go build ./...
GOOS=windows GOARCH=arm64 go build ./...
- name: Vet (windows)
run: GOOS=windows go vet ./...
go vet type-checks _test.go files too, so it also catches Windows-only breaks in the tests (deps_test.go already has runtime.GOOS == "windows" skips). The same command passes locally at 52a5544.
Consider also adding darwin builds, since goreleaser ships those too and the !windows file covers darwin and linux.
Out of scope: actually running the tests on Windows (tracked separately if we do it).
- Lenguaje dominante
- Go
- Estrellas
- 34
- Forks
- 5
- Merge medio
- 5 h 7 min
- PR fusionados (30 d)
- 6
Preparar el entorno
- Sin Dockerfile ni archivo de Docker Compose
- Sin plantilla de pull request
- Leer la guía de contribución
Primeros pasos
- Lee el issue completo y luego la guía de contribución del proyecto.
- Comenta en el issue que vas a ocuparte — evita que dos personas hagan lo mismo.
- Haz un fork del repositorio y trabaja en una rama.
- Abre un pull request que haga referencia al número del issue.
Más de stacklok/modelith
-
bug
Dificultad 2/5 1-3 horas Aptitud para principiantes 86/100
-
documentation
Dificultad 2/5 1-3 horas Aptitud para principiantes 92/100
-
enhancement
Dificultad 2/5 Medio día Aptitud para principiantes 68/100
-
Dificultad 5/5 Más de una semana Aptitud para principiantes 20/100
-
Dificultad 5/5 Más de una semana Aptitud para principiantes 20/100
Todos los issues de stacklok/modelith
Issues similares
-
Dificultad 2/5 1-3 horas Aptitud para principiantes 78/100
-
[开源推荐] FCaptcha：可自行部署的开源验证码Abierto
Dificultad 1/5 Menos de una hora Aptitud para principiantes 65/100
521xueweihan/HelloGitHub#3789 ·
-
Dificultad 2/5 1-3 horas Aptitud para principiantes 88/100
Los mantenedores suelen responder en 12 días
-
stage-fail
Dificultad 2/5 1-3 horas Aptitud para principiantes 78/100
siyuan-note/bazaar#2282 ·
Los mantenedores suelen responder en 1 día
-
Dificultad 2/5 1-3 horas Aptitud para principiantes 78/100
openshift/kube-compare#307 ·
Los mantenedores suelen responder en 1 día