Skip to content

[RR] Reenable forced FIPS checking.#52

Merged
PlaidCat merged 1 commit into
mainlinefrom
{jmaple}_force_fips_check
Dec 22, 2025
Merged

[RR] Reenable forced FIPS checking.#52
PlaidCat merged 1 commit into
mainlinefrom
{jmaple}_force_fips_check

Conversation

@PlaidCat

Copy link
Copy Markdown
Collaborator

A previous commit reverted the if fips check removal. Reverting that additionaly if, and leaving the rest alone.

A previous commit reverted the `if fips` check removal.  Reverting that
additionaly if, and leaving the rest alone.
@PlaidCat PlaidCat self-assigned this Dec 22, 2025
Copilot AI review requested due to automatic review settings December 22, 2025 18:55

Copilot AI left a comment

Copy link
Copy Markdown

Choose a reason for hiding this comment

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

Pull request overview

This PR re-enables forced FIPS checking by removing the conditional if "fips" in rolling_product: check that was previously gating the FIPS protected changes validation. The change ensures that FIPS protected directory checks are performed unconditionally for all rolling release updates, rather than only for products with "fips" in their name.

  • Removes the conditional check that limited FIPS validation to specific products
  • FIPS checking now runs for all rolling release updates regardless of product name

💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.

@bmastbergen bmastbergen left a comment

Copy link
Copy Markdown
Collaborator

Choose a reason for hiding this comment

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

🥌

@PlaidCat PlaidCat merged commit ce5be31 into mainline Dec 22, 2025
7 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Development

Successfully merging this pull request may close these issues.

3 participants