architecture

Please reply the tasks you want to complete below, and I will assign them to you.
请在下方回复你要完成的任务，我会分配给你.

• #1981 @bingxindan
• #1967 @imzhongqi
• #1966 @Ccheers
• #1881 @pwli0755

Is there an existing request for this?

• I have searched the existing requests

Forums discussion

gitter

Subproject(s) affected?

Path

Idea description

Per LGTM, many PathWB conditional DEBUG rules are "unreachable".

"Unreachable code makes the code more difficult to understand and may slow down loading of modules."

See https://lgtm.com/projects

This will help be explicit about which SDK version is expected, and should make CI/CD builds behave more consistently.

https://github.com/kgrzybek/modular-monolith-with-ddd/blob/54d209639e501301e879aa0d0a84c9ea49e4689e/src/Modules/Meetings/Domain/Meetings/Rules/MeetingAttendeesLimitMustBeGreaterThanGuestsLimitRule.cs#L1-L21

I believe the comparison operator used in the IsBroken method is wrong because the rule is checking that meeting attendees limit must be greater than guests limit.