Skip to content

[WIP] Fix code according to review comments#379

Closed
zdz with Copilot wants to merge 2 commits into
masterfrom
copilot/fix-comments-in-review-thread
Closed

[WIP] Fix code according to review comments#379
zdz with Copilot wants to merge 2 commits into
masterfrom
copilot/fix-comments-in-review-thread

Conversation

Copilot AI commented Jul 11, 2026

Copy link
Copy Markdown
Contributor
  • Understand review comments on PR feat(email): 新增可选的 from 字段用于设置发件地址 #378
  • Fix email.rs: add from: Option<String> field; treat empty/whitespace as unconfigured; replace .unwrap() and .expect() with ?/.context() to propagate parse errors
  • Fix config.toml: update comment to clarify that the field must be absent (not empty) to use username as fallback

Copilot AI assigned Copilot and zdz Jul 11, 2026
Copilot AI requested review from Copilot and removed request for Copilot July 11, 2026 14:54
- Add `from: Option<String>` to email Config struct
- Treat empty/whitespace `from` as unconfigured, falling back to `username`
- Replace `.unwrap()` with `?` on mailbox parse to propagate errors via Result
- Update config.toml comment: clarify field must be absent (not empty) to use username fallback
Copilot AI requested review from Copilot and removed request for Copilot July 11, 2026 15:00
@zdz zdz closed this Jul 11, 2026
Copilot AI requested a review from zdz July 11, 2026 15:02
Copilot stopped work on behalf of zdz due to an error July 11, 2026 15:02
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