Skip to content

Bandit has no {col} tag#16336

Closed
dciborow wants to merge 1 commit intomicrosoft:mainfrom
dciborow:patch-1
Closed

Bandit has no {col} tag#16336
dciborow wants to merge 1 commit intomicrosoft:mainfrom
dciborow:patch-1

Conversation

@dciborow
Copy link
Copy Markdown

The col value is being printed into the output for bandit.

##########Linting Output - bandit##########
102,col,LOW,B101:Use of assert detected. The enclosed code will be removed when compiling to optimised byte code.

image

Bandit Documentation
image

The col value is being printed into the output for bandit.

##########Linting Output - bandit##########
102,col,LOW,B101:Use of assert detected. The enclosed code will be removed when compiling to optimised byte code.
@tonybaloney
Copy link
Copy Markdown

bandit does have a col tag, the project just hasn't shipped a release in over 6 months. A workaround is to install bandit from master, but there needs to be a better fix to support both cases.

@tonybaloney
Copy link
Copy Markdown

Have started #16337 to find a permanent fix for this

@karrtikr
Copy link
Copy Markdown

Closing in favor of #16337 which provides fixes for both old and new versions. Thanks for attempting the fix anyways!

@karrtikr karrtikr closed this May 28, 2021
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

Successfully merging this pull request may close these issues.

3 participants