Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Fix issue with list parsing in configuration #784

Merged
merged 3 commits into from May 18, 2019

Conversation

@lossyrob
Copy link
Member

commented May 16, 2019

Overview

Fixes #782

Depends on #785

Notes

This changes the way files and modules are listed in the configuration. It is backwards compatible with the old way we were doing it, which didn't work with lists of more than one value. The correct way to list multiple modules is now e.g.:

modules = tests.test_plugin,tests.test_plugin_2
@lossyrob lossyrob added the review label May 16, 2019
docs/setup.rst Outdated Show resolved Hide resolved
docs/changelog.rst Outdated Show resolved Hide resolved
@lossyrob lossyrob force-pushed the lossyrob:rde/fix/plugin-config branch from 26e71ba to b57bc73 May 17, 2019
@lossyrob lossyrob requested a review from lewfish May 17, 2019
@lossyrob lossyrob force-pushed the lossyrob:rde/fix/plugin-config branch from b57bc73 to 9b00780 May 17, 2019
@lossyrob lossyrob merged commit c0f5cb7 into azavea:master May 18, 2019
1 check passed
1 check passed
continuous-integration/travis-ci/pr The Travis CI build passed
Details
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
2 participants
You can’t perform that action at this time.