Skip to content

Conversation

@cloudstaskdev
Copy link

No description provided.

@LyVanBong
Copy link
Owner

Xin chào — cảm ơn đã gửi PR. Trước khi merge, vui lòng sửa các điểm sau:

  1. Thay các placeholder env bằng giá trị thực (ví dụ: Solution_Name: MyApp.sln, Test_Project_Path: tests/MyApp.Tests/MyApp.Tests.csproj, Wap_Project_Directory và Wap_Project_Path tương ứng).
  2. PR này từ fork nên workflow không có quyền truy cập secrets → bước giải mã PFX và packaging sẽ fail. Tách hoặc bỏ các bước dùng secret khi chạy trên pull_request. Ví dụ thêm condition:
    if: ${{ github.event_name != 'pull_request' }}
    (hoặc chỉ chạy packaging trên push/dispatch).
  3. Thay dotnet test bằng dotnet test <path-to-test-project> hoặc dotnet test MyApp.sln để tránh nhầm lẫn khi có nhiều project.
  4. Chỉ đóng gói MSIX trên Release và khi không phải PR (ví dụ: if: ${{ matrix.configuration == 'Release' && github.event_name != 'pull_request' }}).
  5. Trước khi xóa file .pfx, kiểm tra tồn tại (tránh lỗi). Không in/log bất kỳ secret nào.
  6. Sửa tiêu đề PR (deskop → desktop) và bổ sung mô tả hướng dẫn cách cấu hình env và secrets.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants