Fix test fixture, README provenance and remaining Windows-only polish

Abierto Apto para principiantes
  • ¿Algo no está bien?
#13 0 comentarios 0 reacciones 0 asignados Ver en GitHub

Nadie ha tomado este issue todavía.

Evaluación

Dificultad
2/5
Tiempo estimado
1-3 horas
Aptitud para principiantes
62/100
Tipo de issue
Error
Claridad
Bien especificado
Estado de actividad
Activo
Stack tecnológico
go
Área
devtools

Línea de trabajo

Empieza con el fixture TestCompareWindowsOnly en plugin/scripts/godebugdiff_test.go, donde winreadlinkvolume se espera como Changed: 26 y debería ser 23. Reproduce el punto 1 comentando c.WindowsOnly = isWindowsOnly(k) en compare() y revisa la URL en el mensaje de fallo. Está listo cuando el fixture verifica 23 con la URL #go-123 y comprueba Before, After y Package, y examples/README.md indica en un solo lugar la toolchain de cada archivo.

Escrito por el modelo de indexación a partir del texto del issue.

Descripción

Follow-up from #12:

  1. TestCompareWindowsOnly (plugin/scripts/godebugdiff_test.go) gives winreadlinkvolume Changed: 26. The real value is 23, so the change gets the URL go.dev/doc/godebug#go-126 instead of #go-123. To reproduce: comment out c.WindowsOnly = isWindowsOnly(k) in compare() and run the test; the failure message prints URL:https://go.dev/doc/godebug#go-126. Use 23. The test also ignores the second value compare() returns and never checks Before, After or Package. Asserting them costs nothing.
  2. The WindowsOnly field comment now says "Windows-specific setting", but isWindowsOnly only checks the "win" name prefix. Make the comment match the rule, or derive the mark from an explicit list.
  3. examples/README.md line 12 still says the real outputs were made with Go 1.27.3, and the new line 24 says cobra.txt was redone with Go 1.27.4. Put the toolchain for each file in one place, for example a column in the table.
  4. Item 3 of #11 is still open, although #12 closes #11: the "No binary imports the package, so no effect" section still adds "(Windows only)". To reproduce: run from 1.22 to 1.23 on a module whose only binary is an empty func main() {}. Also, windowsOnly still uses omitempty while goCommand is always emitted, and the report does not tell Linux-only services that they can ignore the pins for Windows-only settings.

GillMacedo

Lenguaje dominante
Go
Estrellas
0
Forks
3
Merge medio
4 h 55 min
PR fusionados (30 d)
4

Preparar el entorno

Abrir en Codespaces

Inicia el contenedor de desarrollo del proyecto en tu navegador, con tu propia cuenta de GitHub.

Primeros pasos

  1. Lee el issue completo y luego la guía de contribución del proyecto.
  2. Comenta en el issue que vas a ocuparte — evita que dos personas hagan lo mismo.
  3. Haz un fork del repositorio y trabaja en una rama.
  4. Abre un pull request que haga referencia al número del issue.

Más de GillMacedo/godebugdiff

Todos los issues de GillMacedo/godebugdiff

Issues similares

Más issues de Go

Recibe los nuevos issues en tu correo

Un resumen breve de issues de GitHub para principiantes.