analyzer: a //go:build ignore generator file turns surplus off for its package
Los mantenedores suelen responder en 1 día
Nadie ha tomado este issue todavía.
Evaluación
- Dificultad
- 2/5
- Tiempo estimado
- 1-3 horas
- Aptitud para principiantes
- 83/100
- Tipo de issue
- Error
- Claridad
- Bien especificado
- Estado de actividad
- Activo
- Stack tecnológico
- go
- Área
- testing-qa, tooling
Línea de trabajo
Empieza por internal/surplus.go:225 y compara la comprobación del analizador para archivos ignorados con la distinción que shrink ya utiliza. Reproduce el comportamiento con un archivo gen.go marcado con //go:build ignore y package main, y luego ejecuta las pruebas relevantes del analizador o el controlador independiente. El trabajo estará listo cuando el analizador siga informando de la declaración compartida innecesaria con ese archivo adicional, también mediante go vet -vettool.
Escrito por el modelo de indexación a partir del texto del issue.
Descripción
len(pass.IgnoredFiles) > 0 (internal/surplus.go:225) switches the rule off for any build-excluded file. That includes the common gen.go written as //go:build ignore plus package main. unseen() already skips files of another package and reads excluded .go files of the same package by name, so this check only needs to cover files that are not Go. shrink already makes that distinction, so the analyzer and shrink disagree here.
Reproduction
A package with one unneeded //declscope:shared func helper():
| Package | Result |
|---|---|
| Without an extra file | //declscope:shared on helper: no use from another namespace is visible to declscope, exit 3 |
With gen.go (//go:build ignore, package main) |
No output, exit 0. The same under the standalone driver and under go vet -vettool |
Suggested fix
len(pass.OtherFiles) > 0 || slices.ContainsFunc(pass.IgnoredFiles, func(f string) bool { return !strings.HasSuffix(f, ".go") })
- Lenguaje dominante
- Go
- Estrellas
- 42
- Forks
- 1
- Merge medio
- 57 min
- PR fusionados (30 d)
- 135
Preparar el entorno
Este proyecto no incluye contenedor de desarrollo, Dockerfile ni guía de contribución, así que la configuración corre por tu cuenta: empieza por su README y consulta nuestra guía para la primera contribución para los pasos generales.
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.
Issues similares
-
Dificultad 2/5 1-3 horas Aptitud para principiantes 74/100
JakeChampion/lang#13132 ·
Los mantenedores suelen responder en 1 día
-
[BUG] deploy.restart_policy without condition drops the restart policyPosiblemente ocupada @akshat-lakhera la tomó hoy. Abierto
Dificultad 2/5 1-3 horas Aptitud para principiantes 82/100
docker/compose#14383 · 1 comentario ·
Los mantenedores suelen responder en 1 día
-
Dificultad 1/5 Menos de una hora Aptitud para principiantes 72/100
Azure/azure-storage-fuse#2429 ·
Los mantenedores suelen responder en 1 día
-
Dificultad 2/5 1-3 horas Aptitud para principiantes 65/100
AvengeMedia/dank-greeter#39 · 1 comentario ·
-
bug: dataplane client ignores SERVER_BASEPATH, so create/delete topic miss the subpathPosiblemente ocupada @bhataprameya la tomó hoy. Abierto
Dificultad 2/5 1-3 horas Aptitud para principiantes 76/100
redpanda-data/console#2693 ·
Los mantenedores suelen responder en 2 días