Skip to content

fix: sequential QA follow-up repairs#2

Merged
jasper0507 merged 3 commits into
mainfrom
fix/qa-followup-sequential-commits
Apr 26, 2026
Merged

fix: sequential QA follow-up repairs#2
jasper0507 merged 3 commits into
mainfrom
fix/qa-followup-sequential-commits

Conversation

@jasper0507
Copy link
Copy Markdown
Owner

变更顺序

  1. 28da2f8 test: repair backend auth regression fixtures
    • 修复后端测试夹具中的失效密码,恢复 go test ./...
  2. c02f420 ci: add manual trigger and explicit permissions
    • 为 CI 增加 workflow_dispatch
    • 显式声明 contents: read 权限,便于排查和手动触发
  3. a3b6a0d fix: show auth-specific error messages
    • 注册时区分邮箱格式/密码长度错误与邮箱已存在
    • 登录时区分账号密码错误
    • 补充前端测试覆盖这些错误提示

验证

  • docker run --rm -v /opt/brev-diary/backend:/app -w /app golang:1.26 go test ./...
  • cd frontend && npm test
  • cd frontend && npm run build

备注

  • 这次按步骤拆成独立 git commit,便于以后逐步回溯
  • 推送后可以直接用这个分支观察 GitHub Actions 是否开始产出检查结果

@jasper0507 jasper0507 merged commit cdb1518 into main Apr 26, 2026
6 checks passed
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.

1 participant