DOI-USGS / DOI-USGS/dataretrieval-python

project-root inference in your code-health workflow

Offen
#415 1 Kommentar 1 Reaktion 0 zugewiesene Personen Auf GitHub ansehen
Vorherrschende Sprache
Python
Sterne
265
Forks
63
Ø Merge
1 T. 20 Std.
Gemergte PRs (30 T.)
19

Beschreibung

Hi! I'm the author of [pyscn](https://github.com/ludo-technologies/pyscn). I came across this repo and noticed you're running it in your weekly code-health workflow. Thanks for using it.

I also read the comment in `code-health.yml` about pyscn inferring the wrong project root and silently resolving only part of the imports, which inflates the score. That's a real problem on my side and I'd like to fix it rather than have you work around it. If you remember what the wrong root looked like (or a run where the sanity check fired), I'd appreciate the details. Same for having to run `analyze` twice to get both JSON and HTML; that should be a single invocation.

Separately, there's a GitHub App that runs the same analysis on a schedule and posts the reports for you, in case it saves some upkeep:
https://github.com/apps/polyscan-app

If anything else comes up, feel free to let me know.

Beitragsleitfaden

Beitragsleitfaden öffnen

Rechercherichtung

Review the comment and current invocation in code-health.yml, then gather the wrong project-root details or a run where the sanity check fired. The requested outcome is for pyscn to resolve the full project correctly and produce both JSON and HTML from one analyze invocation, but the implementation location and scope are not identified.

Vom Indexierungsmodell aus dem Issue-Text verfasst.

Bewertung

Tech-Stack
python
Bereich
tooling
Issue-Typ
Bug
Schwierigkeit
5/5
Geschätzter Aufwand
Über eine Woche
Aktivitätsstatus
Aktiv
Klarheit
Muss geklärt werden
Anfängerfreundlichkeit
25/100

Neue Issues direkt in Ihr Postfach

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