Uh oh!
There was an error while loading.Please reload this page.
- Notifications
You must be signed in to change notification settings - Fork939
Commita6a2a37
committed
Make installation instructions more consistent
If people who want to run the tests didn't install the test extra,they can still install that extra. This simplifies the instructionsaccordingly. test-requirements.txt is still mentioned near thebeginning in case people want to look at it to see dependencies.But the changed code is the only place where the instructions hadstill said to do anything with those files.A possible disadvantage of this change is that in the rare casethat someone following those instructions to run the tests locallydidn't do an editable installation, then installing with the extrashouldn't be done editably either. But this doesn't seem like alikely problem, and the new text has an example command with -e toclarify the kind of command whose effect is augmented here.Because of that subtlety, it is not obvious that this change isreally justified for purposes of clarity. However, this also helpsprepare for a time when test-requirements.txt won't exist anymore,as may happen when the project definition is made fully declarative(see discussion in comments in#1716).1 parent2775b60 commita6a2a37
1 file changed
+2
-5
lines changedLines changed: 2 additions & 5 deletions
Original file line number | Diff line number | Diff line change | |
---|---|---|---|
| |||
145 | 145 |
| |
146 | 146 |
| |
147 | 147 |
| |
148 |
| - | |
149 |
| - | |
150 |
| - | |
151 |
| - | |
152 |
| - | |
| 148 | + | |
| 149 | + | |
153 | 150 |
| |
154 | 151 |
| |
155 | 152 |
| |
|
0 commit comments
Comments
(0)