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.

Sign up for GitHub

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

Jump to bottom

Don't use `assert` outside tests #50

Merged

elliot-100 merged 2 commits into main from 45-dont-use-assert-outside-tests

Sep 21, 2023

Owner

elliot-100 commented Sep 21, 2023

No description provided.

elliot-100 added 2 commits

September 21, 2023 12:07


          Eliminate assert in production code

9da76a6


          Ruff ruleset: add flake8-bandit, but allow assert in tests

bc611bb

elliot-100 linked an issue

that may be closed by this pull request

Don't use assert outside tests #45

Closed

elliot-100 enabled auto-merge

September 21, 2023 11:08

elliot-100 merged commit 6ab46d7 into main

4 checks passed

elliot-100 deleted the 45-dont-use-assert-outside-tests branch

September 21, 2023 11:09

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet