Dry runs of release-pr and prerelease edit the checkout

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

Maintainer thường phản hồi trong vòng 1 ngày

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

Đánh giá

Độ khó
2/5
Thời gian dự kiến
1-3 giờ
Mức phù hợp với người mới
62/100
Loại issue
Lỗi
Độ rõ ràng
Khá rõ ràng
Mức độ hoạt động
Sôi nổi
Công nghệ
bun, git, typescript
Lĩnh vực
devtools, release

Hướng nghiên cứu

Bắt đầu với các kiểm tra chạy thử trong src/release-pr/index.ts quanh dòng 162–166 và src/prerelease/index.ts quanh dòng 97; lần theo checkout, các cập nhật phiên bản và những script release chạy trước đó. Quyết định cách để các lần chạy thử tránh sửa đổi checkout, hoặc liệu README có nên hướng dẫn người dùng dùng một bản clone dùng một lần hay không. Hoàn tất khi một lần chạy thử cục bộ không làm thay đổi checkout hoặc README ghi rõ cách khắc phục tạm thời.

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

Mô tả

bug
Issue description

The README recommends trying an action locally as a dry run (INPUT_DRY_RUN=true bun dist/release-pr/index.js). Before it checks dry, release-pr runs git checkout -B <release branch>, rewrites every selected package.json version and runs each package's release script, which writes its CHANGELOG.md (src/release-pr/index.ts#L162-L166). prerelease also calls setVersion before its dry check (src/prerelease/index.ts#L97).

So a local dry run leaves the developer on a new branch with modified files. Traced, not run (to keep the checkout clean). In CI it doesn't matter.

Expected: a dry run doesn't touch the checkout, or the README says to run it in a throwaway clone.

Found by a read-only audit of main at 435a16b (first done at 046494f; no src/ file changed in between), report audits/actions-2026-11-11.md in the workspace repo.

Code sample
await checkoutBranch(branch);
await applyVersions(group, plans);

let url = "";
if (dry) {
  info(`[DRY] Would push "${branch}" and open the PR "${message}"`);
Versions
  • @nanoforge-dev/actions 3.3.0 (main at 435a16b)
  • Bun 1.3.14
Issue priority

Low (slightly annoying)

I have tested this issue on a development release

435a16b

Ngôn ngữ chính
TypeScript
Star
0
Fork
0
Merge trung bình
15 giờ 51 phút
Pull request đã merge (30 ngày)
21

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

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 NanoForge-dev/actions

Tất cả issue của NanoForge-dev/actions

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.