-
Notifications
You must be signed in to change notification settings - Fork 378
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
fix: Don't show replaced, expired or otherwise abandoned docs for AD #5162
Conversation
Codecov Report
@@ Coverage Diff @@
## main #5162 +/- ##
==========================================
- Coverage 88.54% 88.53% -0.01%
==========================================
Files 295 295
Lines 40046 40058 +12
==========================================
+ Hits 35458 35467 +9
- Misses 4588 4591 +3
Help us with your feedback. Take ten seconds to tell us how you rate us. Have a feature suggestion? Share it here. |
@rjsparks better? |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Yes, this is what you want. I do think it would be better to unwrap the logic puzzle into something that requires less thinking when the next coder gets here. I've suggested an alternative.
Co-authored-by: Robert Sparks <[email protected]>
Fixes #5153 and #2889.