Skip to content

Conversation

@lucanovera
Copy link
Contributor

This reverts commit 71a325f.

Ticket []

Description Of Changes

Code Changes

Steps to Confirm

Pre-Merge Checklist

  • Issue requirements met
  • All CI pipelines succeeded
  • CHANGELOG.md updated
    • Add a db-migration This indicates that a change includes a database migration label to the entry if your change includes a DB migration
    • Add a high-risk This issue suggests changes that have a high-probability of breaking existing code label to the entry if your change includes a high-risk change (i.e. potential for performance impact or unexpected regression) that should be flagged
    • Updates unreleased work already in Changelog, no new entry necessary
  • UX feedback:
    • All UX related changes have been reviewed by a designer
    • No UX review needed
  • Followup issues:
    • Followup issues created
    • No followup issues
  • Database migrations:
    • Ensure that your downrev is up to date with the latest revision on main
    • Ensure that your downgrade() migration is correct and works
      • If a downgrade migration is not possible for this change, please call this out in the PR description!
    • No migrations
  • Documentation:
    • Documentation complete, PR opened in fidesdocs
    • Documentation issue created in fidesdocs
    • If there are any new client scopes created as part of the pull request, remember to update public-facing documentation that references our scope registry
    • No documentation updates required

@vercel
Copy link

vercel bot commented Dec 3, 2025

The latest updates on your projects. Learn more about Vercel for GitHub.

2 Skipped Deployments
Project Deployment Preview Comments Updated (UTC)
fides-plus-nightly Ignored Ignored Dec 3, 2025 9:33pm
fides-privacy-center Ignored Ignored Dec 3, 2025 9:33pm

@greptile-apps
Copy link
Contributor

greptile-apps bot commented Dec 3, 2025

Greptile Overview

Greptile Summary

This PR reverts the 2.57.2 changelog entry that was added in commit 71a325f. The revert is necessary because version 2.57.2 was never actually released as a separate version. The NextJS and React upgrade referenced in the reverted changelog entry (PR #7061) was already included in version 2.58.0 (as noted on line 943 of the changelog).

  • Removed the 2.57.2 changelog section that claimed to fix a React Server vulnerability
  • Corrected the 2.58.0 version comparison link to point to 2.57.1 instead of the non-existent 2.57.2

This is a straightforward revert to correct the versioning history.

Confidence Score: 5/5

  • This PR is completely safe to merge - it's a simple revert of incorrect changelog documentation
  • This receives a perfect score because it's a straightforward documentation revert that corrects an error in the changelog. The change only affects the CHANGELOG.md file, removes a duplicate/incorrect version entry, and fixes the version comparison link. No code changes, no logic changes, no dependencies affected - purely correcting version history documentation.
  • No files require special attention

Important Files Changed

File Analysis

Filename Score Overview
CHANGELOG.md 5/5 Reverted the 2.57.2 changelog entry and corrected the 2.58.0 comparison link to point back to 2.57.1 instead of 2.57.2

Copy link
Contributor

@greptile-apps greptile-apps bot left a comment

Choose a reason for hiding this comment

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

1 file reviewed, no comments

Edit Code Review Agent Settings | Greptile

@NevilleS NevilleS added this pull request to the merge queue Dec 3, 2025
Merged via the queue into main with commit 49fbbb2 Dec 3, 2025
41 checks passed
@NevilleS NevilleS deleted the revert-update-changelog-release-2.57.2 branch December 3, 2025 22:38
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