Skip to content

Navigation Menu

Sign in
Appearance settings

Search code, repositories, users, issues, pull requests...

Provide feedback

We read every piece of feedback, and take your input very seriously.

Saved searches

Use saved searches to filter your results more quickly

Appearance settings

Fix Ruff E501 failures in test_veracrypt assertions#38

Merged
srichs merged 1 commit intomainsrichs/python-veracrypt:mainfrom
codex/fix-ruff-ci-checks-for-line-lengthsrichs/python-veracrypt:codex/fix-ruff-ci-checks-for-line-lengthCopy head branch name to clipboard
Feb 14, 2026
Merged

Fix Ruff E501 failures in test_veracrypt assertions#38
srichs merged 1 commit intomainsrichs/python-veracrypt:mainfrom
codex/fix-ruff-ci-checks-for-line-lengthsrichs/python-veracrypt:codex/fix-ruff-ci-checks-for-line-lengthCopy head branch name to clipboard

Conversation

@srichs
Copy link
Owner

@srichs srichs commented Feb 14, 2026

Motivation

  • Two assertion lines in tests/test_veracrypt.py exceeded the Ruff line-length limit (E501), which caused CI lint failures; the goal is to resolve the lint errors without changing test behavior.

Description

  • Wrap the two long self.assertIn(...) lines in tests/test_veracrypt.py so the message and exception string are split across lines, preserving the original assertion messages and semantics.

Testing

  • Ran ruff check tests/test_veracrypt.py, which completed successfully and reported no issues.

Codex Task

@srichs srichs merged commit dcf49a8 into main Feb 14, 2026
8 of 9 checks passed
@srichs srichs deleted the codex/fix-ruff-ci-checks-for-line-length branch February 14, 2026 08:29
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant

Morty Proxy This is a proxified and sanitized view of the page, visit original site.