Closed ddabble closed 1 year ago
Merging #649 (cb30aa5) into dev (5d12ebc) will increase coverage by
0.05%
. The diff coverage is100.00%
.
@@ Coverage Diff @@
## dev #649 +/- ##
==========================================
+ Coverage 87.39% 87.44% +0.05%
==========================================
Files 135 135
Lines 5719 5744 +25
==========================================
+ Hits 4998 5023 +25
Misses 721 721
Impacted Files | Coverage Δ | |
---|---|---|
src/announcements/urls.py | 100.00% <ø> (ø) |
|
src/checkin/urls.py | 100.00% <ø> (ø) |
|
src/contentbox/models.py | 78.78% <ø> (ø) |
|
src/contentbox/tests/urls/urls_internal.py | 100.00% <ø> (ø) |
|
src/docs/urls.py | 100.00% <ø> (ø) |
|
src/groups/urls.py | 100.00% <ø> (ø) |
|
src/internal/urls.py | 100.00% <ø> (ø) |
|
src/make_queue/urls.py | 100.00% <ø> (ø) |
|
src/makerspace/urls.py | 100.00% <ø> (ø) |
|
src/news/urls.py | 100.00% <ø> (ø) |
|
... and 14 more |
Merging without explicit approval from another member, as the Dev committee previously agreed to merge these changes and assume they're relatively bug-free, simply to get things done quicker.
Proposed changes
Fixes
Other changes
add
(instead ofcreate
) andchange
(instead ofedit
orupdate
), to comply with the style guide (542614586ef5e04b661680d722f4b0a0a7e7fa5d)Areas to review closely
"Add"/"change" verbs I might have missed, and (hardcoded) URLs I might have forgotten to update after the renaming.
Checklist
(If any of the points are not relevant, mark them as checked)