-
Notifications
You must be signed in to change notification settings - Fork 919
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
Waffles Part 2: Move waffle switches to django-waffle 🧇 #15229
Conversation
Codecov ReportAll modified and coverable lines are covered by tests ✅
Additional details and impacted files@@ Coverage Diff @@
## main #15229 +/- ##
==========================================
+ Coverage 77.77% 78.41% +0.64%
==========================================
Files 164 162 -2
Lines 8462 8350 -112
==========================================
- Hits 6581 6548 -33
+ Misses 1881 1802 -79 ☔ View full report in Codecov by Sentry. |
2336f0b
to
ba611bd
Compare
@stevejalim, @pmac Flagged for review, but probably best to review by commit since this branch is based on the part-1 branch. So you'll see both in the "Files changed" view. |
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.
🧇
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.
r+wc
a92e2fc
to
23e5aa9
Compare
23e5aa9
to
2c31e4e
Compare
One-line summary
See #15195 for Part 1.
This is Part 2 after Part 1 is deployed and the import has been run.
Significant changes and points to review
Issue / Bugzilla link
#15157
Testing