完成 MR 通知与 Hook/Channel 模块重构 - #30
Merged
Merged
Conversation
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
变更目的
为 MR 合并、审批通知建立独立的 notifications、channels、hooks、delivery 模块边界,并完成 Issue、Merge Request、Note Webhook Handler 的目录归一化。移除所有旧兼容 facade,使职责模块成为唯一代码入口。
Task 与 Commit
1745ca2ee8009599f07f87919b88283d11cf9a00fa558ae93a5fbc7d6016792db1188aed36f61a284d130f3a85108e56245b707610a7157978d040529a89a09a25078336650965a606950433d5918af4b1143db8cc68c01966d6fde927f1be1c2d59e293943ef78d77bcf174394837d75508a2f5175cf1ea另包含已完成的 MR 合并通知功能基线提交:
d8653d0f36268e56188b1e8040f834469ec4267b16dacabd89553787cb7a6922f22ab20cf2fd6abd验证
75 passed, 1 deselectedgit diff --check通过兼容性与影响
main。远端为无中间提交的最终树提交
a97d0d369dc263b52516fec69d21d792badeadf1,对应本地已验证的完整变更。