Hacktoberfest 2026: le issue che i maintainer hanno segnato per ottobre, aperte e adatte ai principianti. Sfoglia le issue Hacktoberfest

pnpm の版の宣言を packageManager のままにするか、devEngines.packageManager へ移すかを決める

Aperta
  • Qualcosa non va?
#91 0 commenti 0 reazioni 0 assegnatari Vedi su GitHub

Nessuno ha ancora preso questa issue.

Valutazione

Difficoltà
5/5
Tempo stimato
Più di una settimana
Idoneità per principianti
35/100
Tipo di issue
Refactoring
Chiarezza
Abbastanza chiara
Stato di attività
Attiva
Stack tecnologico
json, yaml
Ambito
build-system

Direzione di ricerca

Start by inspecting package.json, pnpm-lock.yaml, and the repository workflows to confirm how the current package-manager declaration is used. Run the documented pnpm install comparison and check whether the lockfile header changes when using devEngines.packageManager. Done means recording the decision and, if migration is chosen, updating the declaration and any resulting lockfile changes.

Scritto dal modello di indicizzazione a partire dal testo della issue.

Descrizione

enhancement

今の状態

このリポジトリは pnpm の版を package.json の packageManager で宣言している。

https://pnpm.io/package_json は packageManager を legacy と呼び、
devEngines.packageManager を案内している。

pnpm を 12.5.1 へ上げる変更では、この宣言を移さなかった。移さない理由は、その
変更の範囲を版の更新に限るためであって、packageManager のほうが優れているから
ではない。両者を分ける実測上の根拠は、下記のとおり見つかっていない。

確かめたこと

版を 1 つに固定できるかどうかは差にならない。使い捨てのプロジェクトに
devEngines.packageManager を
{"name":"pnpm","version":"12.5.1","onFail":"download"} だけで置いて
pnpm install を実行すると、終了コードは 0 で、生成された pnpm-lock.yaml は
次を記録する。

packageManagerDependencies:
  pnpm:
    specifier: 12.5.1
    version: 12.5.1

公式の逐語も「Unlike the packageManager field, this supports version ranges.」
であり、範囲を「書ける」であって「範囲でしか書けない」ではない。

ロックファイルの見た目も差にならない。同じ実測で、devEngines でも先頭に
packageManagerDependencies と @pnpm/exe.* を含む env lockfile 文書が付いた。

CI も差にならない。pnpm/action-setup の v6.1.0 の README は「Omit version
input to use the version in the packageManager or devEngines.packageManager
field in the package.json」と述べており、どちらでも版を読む。このリポジトリの
ワークフローは version を渡していないので、この経路に当たる。

どこをどう良くできるか

devEngines.packageManager へ移すと、公式が legacy と呼ぶ側から離れられる。
onFail の挙動を宣言でき、必要になったときに範囲でも書ける。

移さない場合は、packageManager を使い続ける判断を、legacy と呼ばれていることを
承知のうえで下したものとして記録しておく必要がある。

いずれにせよ、決めるには上記のほかに次を確かめるとよい。devEngines へ移した
ときに pnpm-lock.yaml の先頭の文書が現在のものと一致するか(差分が出るなら、
その差分はこの移行が持ち込んだものだと分かるようにしておく)。

実施しなかった理由

版を上げる変更の中で宣言の場所も移すと、2 つの決定が 1 つのコミットに混ざる。
どちらが原因で CI の挙動が変わったのかを切り分けられなくなる。

Lingua principale
TypeScript
Stelle
0
Fork
0
Nessuna PR unita negli ultimi 30g

Preparare l'ambiente

  • Include un Dockerfile o un file Docker Compose
  • Nessun modello di pull request
  • Nessuna guida per i contributori

Come iniziare

  1. Leggi tutta la issue e poi la guida ai contributi del progetto.
  2. Commenta sulla issue per dire che te ne occupi tu — evita che due persone facciano lo stesso lavoro.
  3. Fai un fork del repository e lavora su un branch.
  4. Apri una pull request che faccia riferimento al numero della issue.

Altre issue di traPtitech/Checkin

Tutte le issue di traPtitech/Checkin

Issue simili

Altre issue su TypeScript

Ricevi le nuove issue nella tua casella

Un breve riepilogo di issue GitHub adatte ai principianti.