Skip to content

Comments

refactor: Move Config related back to config.rs instead#6667

Merged
Xuanwo merged 4 commits intomainfrom
cleanup-config
Oct 14, 2025
Merged

refactor: Move Config related back to config.rs instead#6667
Xuanwo merged 4 commits intomainfrom
cleanup-config

Conversation

@Xuanwo
Copy link
Member

@Xuanwo Xuanwo commented Oct 14, 2025

Which issue does this PR close?

None

Rationale for this change

Code cleanup, no other meaningful changes.

What changes are included in this PR?

Are there any user-facing changes?

This PR was primarily authored with Codex using GPT-5-Codex and then hand-reviewed by me. I AM responsible for every change made in this PR. I aimed to keep it aligned with our goals, though I may have missed minor issues. Please flag anything that feels off, I'll fix it quickly.

Signed-off-by: Xuanwo <[email protected]>
Signed-off-by: Xuanwo <[email protected]>
Signed-off-by: Xuanwo <[email protected]>
@Xuanwo Xuanwo requested review from PsiACE and tisonkun October 14, 2025 12:25
@dosubot dosubot bot added the size:XL This PR changes 500-999 lines, ignoring generated files. label Oct 14, 2025
Signed-off-by: Xuanwo <[email protected]>
@dosubot dosubot bot added the releases-note/refactor The PR does a refactor on code or has a title that begins with "refactor" label Oct 14, 2025
@Xuanwo Xuanwo merged commit d74c8d7 into main Oct 14, 2025
397 of 400 checks passed
@Xuanwo Xuanwo deleted the cleanup-config branch October 14, 2025 12:52
@dosubot dosubot bot added the lgtm This PR has been approved by a maintainer label Oct 14, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

lgtm This PR has been approved by a maintainer releases-note/refactor The PR does a refactor on code or has a title that begins with "refactor" size:XL This PR changes 500-999 lines, ignoring generated files.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants