bazel-contrib / bazel-contrib/rules_python
Add a new python comment annotation that adds targets to `data`
- Langage dominant
- Starlark
- Étoiles
- 688
- Forks
- 721
- Merge moyen
- 15 h 7 min
- PR mergées (30 j)
- 76
Description
# 🚀 feature request
### Relevant Rules
+ gazelle
### Description
This is the same as #1862, but for the `data` attribute on generated targets.
### Describe the solution you'd like
```python
# foo/bar/baz_test.py
import pathlib
# gazelle:include_data //foo/bar:test_data
TEST_DATA_DIR = pathlib.Path(__file__).parent / "test_data"
def test_main():
assert TEST_DATA_DIR.exists()
```
Running Gazelle generates:
```starlark
py_test(
name = "baz_test",
srcs = ["baz_test.py"],
data = ["//foo/bar:test_data"],
)
```
Note that Gazelle does _**not**_ generate the `test_data` target - it's still on the user to do that themselves.
### Describe alternatives you've considered
The current method is to manually add `data` attributes.
Guide de contribution
Ouvrir le guide de contribution
Piste de recherche
Commencez par lire l’issue associée #1862 et l’implémentation de Gazelle pour les annotations existantes dans les commentaires Python. Suivez l’effet des annotations sur les targets générées, puis ajoutez la gestion correspondante de `# gazelle:include_data` afin que l’attribut `data` contienne le label demandé sans générer la target. Le travail est terminé lorsque l’exemple produit la sortie `py_test` indiquée et que le comportement existant reste intact.
Rédigé par le modèle d'indexation à partir du texte de l'issue.
Évaluation
- Stack technique
- python
- Domaine
- build-system, tooling
- Type d'issue
- Fonctionnalité
- Difficulté
- 3/5
- Temps estimé
- 1-2 jours
- Activité
- À l'abandon
- Clarté
- Plutôt claire
- Accessibilité débutants
- 35/100