Reject deprecated Needs inventory inputs in docs(data)

Abierto
  • ¿Algo no está bien?
#1,018 0 comentarios 0 reacciones 0 asignados Ver en GitHub

Los mantenedores suelen responder en 1 día

Nadie ha tomado este issue todavía.

Evaluación

Dificultad
4/5
Tiempo estimado
3-5 días
Aptitud para principiantes
47/100
Tipo de issue
Refactorización
Claridad
Bien especificado
Estado de actividad
Activo
Stack tecnológico
python
Área
tooling

Línea de trabajo

Start at the public macro boundary in docs.bzl and read the deprecation documented in #897; locate the consumers and fixtures that pass inventory labels through data. Migrate those inputs to external_needs, then trace the interactive build and Needs export paths to identify the implicit imports to remove. Done means inventory labels in data are rejected with a diagnostic, explicit imports and ordinary data files still work, and the API and migration docs plus coverage are updated.

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

Descripción

intentionally-limited

Needs inventories should be imported explicitly through docs(external_needs = [...]). Passing inventory targets through docs(data = [...]) is deprecated but still accepted, leaving two routes for the same dependency and retaining compatibility plumbing.

The deprecation is documented in #897 and reported at the public macro boundary in docs.bzl. Remove this compatibility route in a breaking release.

Acceptance criteria:

  • Migrate remaining consumers and fixtures from inventory labels in data to external_needs.
  • Reject recognized Needs inventory labels in docs(data = [...]) with a clear diagnostic pointing to external_needs.
  • Remove the implicit inventory import from data in both interactive builds and Needs export actions.
  • Preserve ordinary supporting/runtime data files and existing supported external_needs inputs.
  • Cover the rejection and successful explicit import, and update the API and migration documentation.

Keep the scope limited to the deprecated data route. Removing the legacy needs_json label from explicit external_needs is a separate decision. Removing bundle data mounts is also a separate change.

Lenguaje dominante
Python
Estrellas
11
Forks
39
Merge medio
1 d 13 h
PR fusionados (30 d)
46

Preparar el entorno

Abrir en Codespaces

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

  • Sin Dockerfile ni archivo de Docker Compose
  • Tiene una plantilla de pull request
  • Sin guía de contribución

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 eclipse-score/docs-as-code

Todos los issues de eclipse-score/docs-as-code

Issues similares

Más issues de Python

Recibe los nuevos issues en tu correo

Un resumen breve de issues de GitHub para principiantes.