Skip to content

Remove invalid waivers per productization results for 0.1.80.#562

Merged
comps merged 1 commit intoRHSecurityCompliance:mainfrom
ggbecker:remove-invalid-waivers
Mar 6, 2026
Merged

Remove invalid waivers per productization results for 0.1.80.#562
comps merged 1 commit intoRHSecurityCompliance:mainfrom
ggbecker:remove-invalid-waivers

Conversation

@ggbecker
Copy link
Contributor

@ggbecker ggbecker commented Mar 6, 2026

The review was done in this document (restricted access)

I believe the removed permanent waivers for static-checks/html-links can be removed based on latest advancements on how we retrieve information from the web (using curl instead of python requests)

Copy link
Contributor

@comps comps left a comment

Choose a reason for hiding this comment

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

I've provided some comments, the rest of the removals look good, I cross-checked it against a regular daily.

@ggbecker
Copy link
Contributor Author

ggbecker commented Mar 6, 2026

I've provided some comments, the rest of the removals look good, I cross-checked it against a regular daily.

I will try to adjust them and also see if the open issues can actually be closed, I'm closing them as I see fit.

@ggbecker ggbecker force-pushed the remove-invalid-waivers branch from 2c0141d to 94a748e Compare March 6, 2026 16:29
@ggbecker
Copy link
Contributor Author

ggbecker commented Mar 6, 2026

I've updated the PR according to the feedback.

@ggbecker ggbecker force-pushed the remove-invalid-waivers branch from 94a748e to 499a3cd Compare March 6, 2026 16:30
@comps comps merged commit fe0fa58 into RHSecurityCompliance:main Mar 6, 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.

2 participants