Skip to content

Revert "test : added unit tests for FilterPreset and SavedViewCreate Pydantic validators"#1298

Closed
utksh1 wants to merge 2 commits into
mainfrom
revert-1231-#1225
Closed

Revert "test : added unit tests for FilterPreset and SavedViewCreate Pydantic validators"#1298
utksh1 wants to merge 2 commits into
mainfrom
revert-1231-#1225

Conversation

@utksh1

@utksh1 utksh1 commented Jun 25, 2026

Copy link
Copy Markdown
Owner

Reverts #1231

utksh1 and others added 2 commits June 25, 2026 05:46
, #1234, #1232, #1231, #1230, #1229, #1228, #1222, #1221, #1254, #1252, #1251, #1250, #1242, #1255, #1090, #1097, #1112)

Reverting all PR merges performed during this session to restore repository
to pre-session state. This reverts commits:
- e376276 (#1255)
- 18e5101 (#1222)
- e516dd5 (#1230)
- be400fa (#1235)
- d522e9a (#1243)
- bc0b1cd (#1090)
- de56660 (#1097)
- c2a5a53 (#1112)
- eb30d33 (#1254)
- f2a67f7 (#1250)
- 6579112 (#1251)
- 8751d92 (#1252)
- 3ebc935 (#1221)
- 6f5a936 (#1228)
- da15da4 (#1229)
- b66a5b0 (#1231)
- 775c5c2 (#1232)
- b529ddb (#1234)
- 965b001 (#1241)
- d46e573 (#1242)
- dd09955 (#1240)

All PRs are also being reopened and gssoc labels removed.
@utksh1

utksh1 commented Jun 25, 2026

Copy link
Copy Markdown
Owner Author

Closing revert PR. The original code changes have already been reverted on main.

@utksh1 utksh1 closed this Jun 25, 2026
@utksh1 utksh1 reopened this Jun 25, 2026
@utksh1 utksh1 added the gssoc:invalid Admin validation: invalid for GSSoC scoring label Jun 25, 2026
@utksh1

utksh1 commented Jun 25, 2026

Copy link
Copy Markdown
Owner Author

These 22 revert PRs (#1283#1304) are byte-for-byte identical — each carries the exact same 5,299-line diff (+474/−3680 across the same 47 files), despite each claiming to revert a different PR. A genuine git revert <merge-commit> of distinct commits would produce distinct diffs, so these were not generated by reverting the named PRs; they are 22 duplicate branches carrying one shared mass-deletion.

Closing as invalid because:

  • They are duplicates of each other (only one could ever merge; the rest would immediately conflict).
  • The shared diff is destructive: it deletes 12 files (incl. backend/secuscan/rate_limiter.py, the webhook-egress policy integration test, four finding_intelligence unit tests, Reports.onboarding.test.tsx) and strips ~3,680 lines from auth.py, executor.py, routes.py, notification_service.py, and several frontend pages.
  • None corresponds cleanly to the single PR it nominally reverts.

If a rollback of a specific merged PR is actually wanted, please open one PR per merge using git revert <merge-commit> so each diff is scoped and reviewable. Happy to help scope that.

@utksh1 utksh1 closed this Jun 25, 2026
@utksh1 utksh1 added level:beginner 20 pts difficulty label for small beginner-friendly PRs type:refactor Refactor work category bonus label area:backend Backend API, database, or service work labels Jun 25, 2026
@utksh1 utksh1 deleted the revert-1231-#1225 branch June 30, 2026 02:54
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

area:backend Backend API, database, or service work gssoc:invalid Admin validation: invalid for GSSoC scoring level:beginner 20 pts difficulty label for small beginner-friendly PRs type:refactor Refactor work category bonus label

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant