Skip to content

Add three waivers and update RUFF ignore list#539

Merged
matusmarhefka merged 4 commits intoRHSecurityCompliance:mainfrom
ggbecker:add-waiver-12
Feb 13, 2026
Merged

Add three waivers and update RUFF ignore list#539
matusmarhefka merged 4 commits intoRHSecurityCompliance:mainfrom
ggbecker:add-waiver-12

Conversation

@ggbecker
Copy link
Contributor

@ggbecker ggbecker commented Feb 13, 2026

Add three waivers and update RUFF ignore list

See individual commit messages for more details.

@comps
Copy link
Contributor

comps commented Feb 13, 2026

@ggbecker Next time please use human description for PR name / commit header, ie. "waive non-existing /etc/systemd/logind.conf.d".

The ruff error seems bogus, we are not comparing floating points, the 10.2 is interpreted specially by the _Rhel class, so feel free to put RUF069 in the ignores in ruff.yaml.

@ggbecker
Copy link
Contributor Author

Okay, I will add more waivers and fix this one, please wait

@ggbecker ggbecker marked this pull request as draft February 13, 2026 13:02
We are not comparing floating points, the 10.2 in waivers is interpreted specially
by the _Rhel class.
@ggbecker ggbecker changed the title Waiver for https://github.com/ComplianceAsCode/content/issues/14388 Add three waivers and update RUFF ignore list Feb 13, 2026
@ggbecker ggbecker marked this pull request as ready for review February 13, 2026 13:13
Copy link
Contributor

@matusmarhefka matusmarhefka left a comment

Choose a reason for hiding this comment

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

LGTM

@matusmarhefka matusmarhefka merged commit 89b828f into RHSecurityCompliance:main Feb 13, 2026
4 checks passed
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