Skip to content

docs: properly ignore bandit warnings #623

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

Merged
merged 8 commits into from
Aug 5, 2025
Merged

Conversation

clatapie
Copy link
Contributor

This PR explains how to properly ignore bandit warnings.

@clatapie clatapie requested a review from a team July 28, 2025 08:46
@clatapie clatapie self-assigned this Jul 28, 2025
@github-actions github-actions bot added the documentation Improvements or additions to documentation label Jul 28, 2025
Copy link
Contributor

@MaxJPRey MaxJPRey left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM.

@clatapie
Copy link
Contributor Author

clatapie commented Aug 1, 2025

Thanks for your reviews @Revathyvenugopal162 and @MaxJPRey

@clatapie clatapie enabled auto-merge (squash) August 5, 2025 08:05
@clatapie clatapie merged commit 0490e52 into main Aug 5, 2025
8 checks passed
@clatapie clatapie deleted the docs/skip_vulnerabilities branch August 5, 2025 08:14
github-actions bot pushed a commit that referenced this pull request Aug 5, 2025
* docs: adding documentation on how to properly ignore bandit warnings

* fix: typo

* fix: vale issues

* Update doc/source/how-to/vulnerabilities.rst

Co-authored-by: Maxime Rey <[email protected]>

* docs: adding warning section

* fix: adding extra space

* Update doc/source/how-to/vulnerabilities.rst

Co-authored-by: Revathy Venugopal <[email protected]>

---------

Co-authored-by: Maxime Rey <[email protected]>
Co-authored-by: Revathy Venugopal <[email protected]>
Copy link
Member

@RobPasMue RobPasMue left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Much appreciated @clatapie

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
documentation Improvements or additions to documentation
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants