bazel-contrib / bazel-contrib/rules_python

FR: add PyInfo.direct_imports

Offen
#2,942 0 Kommentare 0 Reaktionen 0 zugewiesene Personen Auf GitHub ansehen
core-rules type: feature request
Vorherrschende Sprache
Starlark
Sterne
688
Forks
721
Ø Merge
15 Std. 7 Min.
Gemergte PRs (30 T.)
76

Beschreibung

Currently, the PyInfo's imports field returns all imports paths defined in a target, combined with the ones from its transitive dependencies, as defined here

There's no way to retrieve the direct import paths. It would be useful for IDE support, where we typically read data about a large number of targets and dependencies between them. In that case, we pessimistically would end up in n^2 import entries when reading that, because of depset expansion.

Alternatively IDEs could read just direct_imports and deps fields, and map them to their internal graph. Full transitive imports could still be resolved on demand, based on deps

What underlying problem are you trying to solve with this feature?

Avoid depset flattening when reading both imports and deps from PyInfo, according to https://bazel.build/rules/performance#avoid-depset-to-list

(originally from https://github.com/bazelbuild/bazel/issues/20449 by @tpasternak )

Beitragsleitfaden

Beitragsleitfaden öffnen

Rechercherichtung

Beginne damit, den PyInfo-Provider sowie die vorhandenen Felder imports und deps zu finden, und lies anschließend die verlinkten Bazel-Hinweise zum Vermeiden der depset-to-list-Konvertierung. Als abgeschlossen gilt die Bereitstellung direkter Importpfade, ohne transitive Abhängigkeiten zu flatten, wobei das bestehende Verhalten der transitiven imports erhalten bleibt; im Issue wird keine bestimmte Datei oder kein bestimmter Test genannt.

Vom Indexierungsmodell aus dem Issue-Text verfasst.

Bewertung

Tech-Stack
python
Bereich
build-system
Issue-Typ
Feature
Schwierigkeit
4/5
Geschätzter Aufwand
3-5 Tage
Aktivitätsstatus
Veraltet
Klarheit
Größtenteils klar
Anfängerfreundlichkeit
42/100

Neue Issues direkt in Ihr Postfach

Eine kurze Übersicht über anfängerfreundliche GitHub-Issues.