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

Bug with the whitespace and other delimiters in config #107

Open
vergeev opened this issue Apr 13, 2018 · 0 comments
Open

Bug with the whitespace and other delimiters in config #107

vergeev opened this issue Apr 13, 2018 · 0 comments

Comments

@vergeev
Copy link
Contributor

vergeev commented Apr 13, 2018

It doesn't look like the config parser accounts for the case

directories_to_skip=build, dist, test_fixtures, .pytest_cache

because it doesn't strip whitespace:

'directories_to_skip': lambda x: x.split(',')

(see https://github.com/devmanorg/fiasko_bro/blob/master/fiasko_bro/configparser_helpers.py#L30)

I might also want to be able to do this:

directories_to_skip=
    build
    dist
    test_fixtures
   .pytest_cache

just like in any normal config.

Somebody has probably already written code for that.

@vergeev vergeev changed the title Improve list parameter handling in CLI Bug with the whitespace and other delimiters in config Apr 13, 2018
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

No branches or pull requests

1 participant