Fix syntax of tox.ini.
ClosedPublic
Actions

Authored by vlorentz on Mar 1 2019, 11:37 AM.

Details

Reviewers

olasd

Group Reviewers

Reviewers

Commits

rDOBJS5503845bf200: Make tox install '.[testing]' like other modules, instead of using -r.

Summary

For some reason it did not work for me...

Diff Detail

Repository

rDOBJS Object storage

Lint

Automatic diff as part of commit; lint not applicable.

Unit

Automatic diff as part of commit; unit tests not applicable.

Event Timeline

vlorentz created this revision.Mar 1 2019, 11:37 AM

Herald added a reviewer: Reviewers. · View Herald TranscriptMar 1 2019, 11:37 AM

Build is green
See https://jenkins.softwareheritage.org/job/DOBJS/job/tox/18/ for more details.

Harbormaster completed remote builds in B4520: Diff 3821.Mar 1 2019, 11:39 AM

vlorentz edited the summary of this revision. (Show Details)Mar 1 2019, 12:12 PM

vlorentz edited the summary of this revision. (Show Details)

The change makes sense. However, other modules use .[testing] rather than requirements-test.txt, maybe we should align to that instead?

This revision is now accepted and ready to land.Mar 5 2019, 8:39 AM

Make tox install '.[testing]' like other modules, instead of using -r.

rebase

Closed by commit rDOBJS5503845bf200: Make tox install '.[testing]' like other modules, instead of using -r. (authored by vlorentz). · Explain WhyMar 11 2019, 11:25 AM

This revision was automatically updated to reflect the committed changes.

Harbormaster failed remote builds in B4576: Diff 3869!Mar 11 2019, 11:25 AM

Build has FAILED

Link to build: https://jenkins.softwareheritage.org/job/DOBJS/job/tox/20/
See console output for more information: https://jenkins.softwareheritage.org/job/DOBJS/job/tox/20/console

Harbormaster failed remote builds in B4577: Diff 3870!Mar 11 2019, 11:26 AM

Build has FAILED

Link to build: https://jenkins.softwareheritage.org/job/DOBJS/job/tox/21/
See console output for more information: https://jenkins.softwareheritage.org/job/DOBJS/job/tox/21/console

Revision Contents
Changeset List

Path

Size

tox.ini

2 lines

Diff 3871

View Options

Fix syntax of tox.ini.ClosedPublicActions