-
Notifications
You must be signed in to change notification settings - Fork 316
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
planner: Fix 'nOf' prereq check with module code wildcards #3758
planner: Fix 'nOf' prereq check with module code wildcards #3758
Conversation
The latest updates on your projects. Learn more about Vercel for Git ↗︎
|
@Wxy2003-xy is attempting to deploy a commit to the modsbot's projects Team on Vercel. A member of the Team first needs to authorize it. |
Codecov ReportAttention: Patch coverage is
Additional details and impacted files@@ Coverage Diff @@
## master #3758 +/- ##
==========================================
+ Coverage 53.87% 54.85% +0.97%
==========================================
Files 274 274
Lines 6032 6049 +17
Branches 1449 1446 -3
==========================================
+ Hits 3250 3318 +68
+ Misses 2782 2731 -51 ☔ View full report in Codecov by Sentry. |
cf50cb1
to
4a5ccad
Compare
Thank you. I refactored the code a bit to reduce code duplication and fixed an annoying disambiguity when displaying conflict texts involving multiple 'or', 'and', and 'nOf' fragments. |
Context
#3740 prerequisite check logic for nOf wildcard case not working as intended
Separately, we now also disambiguate conflict texts involving 'and', 'or', and 'nOf' prereq fragments.
Implementation
handle wildcard case in nOf cases separately from normal course code
Other Information