Commit ac2a581
committed
minor #59898 replace
This PR was merged into the 7.3 branch.
Discussion
----------
replace `assertEmpty()` with stricter assertions
| Q | A
| ------------- | ---
| Branch? | 7.3
| Bug fix? | no
| New feature? | no
| Deprecations? | no
| Issues |
| License | MIT
In the code we are strict about not using `empty()`. In my opinion it makes sense to be equally strict with our assertions. But I'd like to gather some feedback first before updating all places.
Commits
-------
6550cb815e3 replace assertEmpty() with stricter assertionsassertEmpty() with stricter assertions (xabbuh)1 file changed
+1
-1
lines changed| Original file line number | Diff line number | Diff line change | |
|---|---|---|---|
| |||
210 | 210 | | |
211 | 211 | | |
212 | 212 | | |
213 | | - | |
| 213 | + | |
214 | 214 | | |
215 | 215 | | |
216 | 216 | | |
| |||
0 commit comments