search

mozilla / bedrock

Making mozilla.org awesome, one pebble at a time
https://www.mozilla.org
Mozilla Public License 2.0
1.18k stars 919 forks source link

Waffles Part 2: Move waffle switches to django-waffle 🧇 #15229

Closed robhudson closed 1 month ago

robhudson commented 1 month ago

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

codecov[bot] commented 1 month ago

Codecov Report

All modified and coverable lines are covered by tests :white_check_mark:

Project coverage is 78.41%. Comparing base (009357a) to head (2c31e4e). Report is 4 commits behind head on main.

Additional details and impacted files ```diff @@ 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 ```

:umbrella: View full report in Codecov by Sentry.
:loudspeaker: Have feedback on the report? Share it here.

robhudson commented 1 month ago

@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.