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

Handle spaces in config options #89

Closed
jakkdl opened this issue Dec 19, 2022 · 2 comments
Closed

Handle spaces in config options #89

jakkdl opened this issue Dec 19, 2022 · 2 comments

Comments

@jakkdl
Copy link
Member

jakkdl commented Dec 19, 2022

There's a bunch of different ways of handling spaces in comma_separated_list to fix --trio200-blocking-calls. Pending on what happens with PyCQA/flake8#1771

@jakkdl
Copy link
Member Author

jakkdl commented Dec 20, 2022

Since 1771 was closed as invalid, this should likely be ignored for --startable-in-context-manager (since spaces aren't valid identifiers anyway), but --trio200-blocking-calls will need a custom type that handles this.

jakkdl added a commit to jakkdl/flake8-async that referenced this issue Dec 29, 2022
… and --trio200-blocking-calls now handle spaces and newlines
jakkdl added a commit to jakkdl/flake8-async that referenced this issue Dec 29, 2022
… and --trio200-blocking-calls now handle spaces and newlines
@Zac-HD
Copy link
Member

Zac-HD commented Dec 31, 2022

Closed by #93. (you need a keyword per issue #, annoyingly)

@Zac-HD Zac-HD closed this as completed Dec 31, 2022
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

2 participants