-
Notifications
You must be signed in to change notification settings - Fork 63
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
feat: removed enable_moderation_reason_codes flag #615
Conversation
8256bbe
to
d57726e
Compare
fix: resolved linter error fix: changed workflow fix: changed workflow fix: changed workflow fix: changed workflow fix: changed workflow fix: changed workflow fix: changed workflow fix: changed workflow fix: changed workflow fix: changed workflow fix: changed workflow fix: changed workflow
894b7c6
to
1c2d4b4
Compare
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.
PR LGTM. @AhtishamShahid please check test cases are failing.
I'll review again once the failed test cases will pass.
Codecov ReportAll modified and coverable lines are covered by tests ✅
Additional details and impacted files@@ Coverage Diff @@
## master #615 +/- ##
==========================================
- Coverage 92.37% 92.36% -0.02%
==========================================
Files 169 169
Lines 3450 3445 -5
Branches 898 897 -1
==========================================
- Hits 3187 3182 -5
Misses 243 243
Partials 20 20 ☔ View full report in Codecov by Sentry. |
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.
PR LGTM
description.
Deprecated enable_moderation_reason_codes flag from the discussion app
Ticket
https://2u-internal.atlassian.net/browse/INF-1159