Skip to content
This repository has been archived by the owner on Jan 10, 2024. It is now read-only.

Clean up usage of raw strings #103

Open
evqna opened this issue Jan 15, 2019 · 3 comments
Open

Clean up usage of raw strings #103

evqna opened this issue Jan 15, 2019 · 3 comments
Labels
bug Something isn't working good first issue Good for newcomers task-size-small
Milestone

Comments

@evqna
Copy link
Contributor

evqna commented Jan 15, 2019

The strings 'bandit' and 'gosec' appear in several places in the code, it may be time to clean that up.

@evqna evqna mentioned this issue Jan 15, 2019
5 tasks
@ericwb ericwb added bug Something isn't working good first issue Good for newcomers labels Jan 17, 2019
@MVrachev
Copy link
Contributor

How do you imagine this cleanup?
You will put a const variable called 'bandit' or 'python_linter' or what?

@evqna
Copy link
Contributor Author

evqna commented Jan 22, 2019

Preferably using an enumeration or some other type-safe construct that prevents mis-typing or introducing bugs by not keeping the different uses in sync

@evqna
Copy link
Contributor Author

evqna commented Jan 22, 2019

A gloal const string could also work but it still requires a bunch of code switches in a few places to call the correct code for each linter.

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
bug Something isn't working good first issue Good for newcomers task-size-small
Projects
None yet
Development

No branches or pull requests

4 participants