Hacktoberfest 2026: những issue maintainer đã đánh dấu cho tháng Mười, đang mở và phù hợp người mới. Xem issue Hacktoberfest

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

Đang mở
  • Có gì không ổn?
#91 0 bình luận 0 reaction 0 người được giao Xem trên GitHub

Chưa có ai nhận issue này.

Đánh giá

Độ khó
5/5
Thời gian dự kiến
Hơn một tuần
Mức phù hợp với người mới
35/100
Loại issue
Tái cấu trúc
Độ rõ ràng
Khá rõ ràng
Mức độ hoạt động
Sôi nổi
Công nghệ
json, yaml
Lĩnh vực
build-system

Hướng nghiên cứu

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.

Do mô hình lập chỉ mục viết ra từ nội dung của issue.

Mô tả

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 の挙動が変わったのかを切り分けられなくなる。

Ngôn ngữ chính
TypeScript
Star
0
Fork
0
Không có pull request nào được merge trong 30 ngày

Chuẩn bị môi trường

  • Có Dockerfile hoặc tệp Docker Compose
  • Không có mẫu pull request
  • Không có hướng dẫn đóng góp

Bắt đầu từ đâu

  1. Đọc hết issue, rồi đọc hướng dẫn đóng góp của dự án.
  2. Bình luận trên issue rằng bạn sẽ nhận — tránh hai người làm cùng một việc.
  3. Fork repository và làm thay đổi trên một nhánh.
  4. Mở pull request có tham chiếu số hiệu của issue.

Issue khác của traPtitech/Checkin

Tất cả issue của traPtitech/Checkin

Issue tương tự

Thêm issue về TypeScript

Nhận issue mới trong hộp thư của bạn

Bản tóm tắt ngắn những issue GitHub phù hợp với người mới.