spago publish reports used test dependencies as unused when setting pedanticPackages
Valutazione
- Difficoltà
- 3/5
- Tempo stimato
- 1-2 giorni
- Idoneità per principianti
- 68/100
- Tipo di issue
- Bug
- Chiarezza
- Specificata chiaramente
- Stato di attività
- Attiva
- Ambito
- build-system
Direzione di ricerca
Start by tracing the publish path where hasTests is set to false and how package.test.pedanticPackages is checked; the issue does not name specific files or tests. Reproduce with the listed commands on the v6.0.1 tag of purescript-unsafe-coerce. Done when publishing checks production dependencies without flagging the excluded test dependencies, while normal builds and tests still check both.
Scritto dal modello di indicizzazione a partire dal testo della issue.
Descrizione
I found this while releasing unsafe-coerce with Spago 1.0.5 — that library has no production dependencies, but its tests use console, effect, and prelude, with package.test.pedanticPackages: true. Both spago build and spago test pass, but spago publish fails:
Tests for package 'unsafe-coerce' declare unused dependencies - please remove them from the project config:
- console
- effect
- prelude
You can reproduce this from the v6.0.1 tag of purescript/purescript-unsafe-coerce, using Spago 1.0.5:
npm install
npx spago build
npx spago test
npx spago publish --offline
Publishing excludes test sources by setting hasTests to false, but the build still checks test dependencies because package.test.pedanticPackages is enabled. With no test imports in the source graph, it then incorrectly reports every test dependency as unused. Instead, IMO publishing should check production dependencies without checking dependencies of tests it deliberately excluded. Normal builds and tests should continue checking both.
- Lingua principale
- PureScript
- Stelle
- 830
- Fork
- 138
- Merge medio
- 6h 21m
- PR unite (30g)
- 4
Preparare l'ambiente
- Nessun Dockerfile né file Docker Compose
- Ha un modello di pull request
- Leggi la guida per i contributori
Come iniziare
- Leggi tutta la issue e poi la guida ai contributi del progetto.
- Commenta sulla issue per dire che te ne occupi tu — evita che due persone facciano lo stesso lavoro.
- Fai un fork del repository e lavora su un branch.
- Apri una pull request che faccia riferimento al numero della issue.
Altre issue di purescript/spago
-
Difficoltà 1/5 Meno di un'ora Idoneità per principianti 82/100
purescript/spago#1403 ·
-
Difficoltà 3/5 Mezza giornata Idoneità per principianti 50/100
purescript/spago#1427 · 2 commenti · 1 reazione ·
-
Difficoltà 1/5 Meno di un'ora Idoneità per principianti 22/100
purescript/spago#1426 ·
-
bug
Difficoltà 3/5 1-2 giorni Idoneità per principianti 45/100
purescript/spago#1386 · 1 commento ·
-
bug UX
Difficoltà 3/5 1-2 giorni Idoneità per principianti 35/100
purescript/spago#1330 · 2 commenti ·
Tutte le issue di purescript/spago
Issue simili
-
v1.5.0 fails to build with GOPROXY=direct: go.mod requires cel-go v0.31.0, which is not present upstreamForse già presa Una pull request collegata a questa issue è aperta o già unita. Aperta
Difficoltà 2/5 1-3 ore Idoneità per principianti 78/100
kubernetes-sigs/dranet#392 ·
I maintainer di solito rispondono entro 1 giorno
-
Difficoltà 2/5 1-3 ore Idoneità per principianti 68/100
open-government-design-system/ogds-elements#199 ·
I maintainer di solito rispondono entro 1 giorno
-
priority: high tech-debt
Difficoltà 2/5 1-3 ore Idoneità per principianti 68/100
cofade/open-garden-planner#538 ·
I maintainer di solito rispondono entro 1 giorno
-
Difficoltà 2/5 1-3 ore Idoneità per principianti 78/100
conda-forge/staged-recipes#35256 · 1 commento ·
I maintainer di solito rispondono entro 1 giorno
-
good first issue
Difficoltà 2/5 1-3 ore Idoneità per principianti 82/100
IkarusAILabs/OpenPulse#105 ·
I maintainer di solito rispondono entro 1 giorno