Skip to content

Remove plans directory#7951

Merged
galvana merged 1 commit intomainfrom
remove-plans-dir
Apr 16, 2026
Merged

Remove plans directory#7951
galvana merged 1 commit intomainfrom
remove-plans-dir

Conversation

@galvana
Copy link
Copy Markdown
Contributor

@galvana galvana commented Apr 16, 2026

Summary

  • Removes the plans/ directory from the repo

Test plan

  • Verify plans/ directory no longer exists in the repo

🤖 Generated with Claude Code

Co-Authored-By: Claude Opus 4.6 (1M context) <noreply@anthropic.com>
@vercel
Copy link
Copy Markdown
Contributor

vercel Bot commented Apr 16, 2026

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

2 Skipped Deployments
Project Deployment Actions Updated (UTC)
fides-plus-nightly Ignored Ignored Preview Apr 16, 2026 11:09pm
fides-privacy-center Ignored Ignored Apr 16, 2026 11:09pm

Request Review

@galvana galvana changed the title Remove accidentally committed plans directory Remove plans directory Apr 16, 2026
@galvana galvana requested a review from gilluminate April 16, 2026 23:10
Copy link
Copy Markdown
Contributor

@claude claude Bot left a comment

Choose a reason for hiding this comment

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

Code Review: Remove plans/ directory

This is a straightforward cleanup PR — it deletes plans/pbac-ui-data-purposes-consumers.md (659 lines), which contained the technical plan for the PBAC UI Data Purposes & Data Consumers feature.

Assessment

No issues. Removing ad-hoc planning documents from the repo is good hygiene once the work has been captured in Jira tickets or is underway. The file had no runtime impact and removing it eliminates stale documentation from the codebase.

Nothing to flag here — LGTM.

🔬 Codegraph: unavailable


💡 Write /code-review in a comment to re-run this review.

@galvana galvana added this pull request to the merge queue Apr 16, 2026
Merged via the queue into main with commit 795eb71 Apr 16, 2026
45 of 46 checks passed
@galvana galvana deleted the remove-plans-dir branch April 16, 2026 23:37
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