Skip to content

chore: remove temporary plan files that were accidentally merged#28381

Merged
Ryukemeister merged 2 commits intomainfrom
devin/1773261428-remove-plan-files
Mar 12, 2026
Merged

chore: remove temporary plan files that were accidentally merged#28381
Ryukemeister merged 2 commits intomainfrom
devin/1773261428-remove-plan-files

Conversation

@eunjae-lee
Copy link
Contributor

What does this PR do?

Removes two temporary implementation plan files that were accidentally merged into the repository:

  • packages/features/feature-opt-in/FEATURE_OPT_IN_BANNER_PLAN.md — Feature opt-in banner system plan
  • packages/features/flags/DECORATOR_IMPLEMENTATION_PLAN.md — Memoize/Unmemoize decorator implementation plan

These are internal planning documents, not project documentation, and should not live in the codebase.

No functional code changes.

Mandatory Tasks (DO NOT REMOVE)

  • I have self-reviewed the code (A decent size PR without self-review might be rejected).
  • I have updated the developer docs in /docs if this PR makes changes that would require a documentation change. N/A
  • I confirm automated tests are in place that prove my fix is effective or that my feature works. N/A — deletion of non-code files only.

How should this be tested?

No testing needed — this PR only deletes markdown plan files. Verify the two files no longer exist after merge.

Human Review Checklist

  • Confirm no other temporary plan files remain in the repo (I searched for *PLAN*.md and only found these two)
  • Verify these files are not referenced or linked from any code or docs

Requested by: @eunjae-lee
Link to Devin session: https://app.devin.ai/sessions/48e0b613eea64389811af91c09910bd3

Co-Authored-By: eunjae@cal.com <hey@eunjae.dev>
@devin-ai-integration
Copy link
Contributor

🤖 Devin AI Engineer

I'll be helping with this pull request! Here's what you should know:

✅ I will automatically:

  • Address comments on this PR that start with 'DevinAI' or '@devin'.
  • Look at CI failures and help fix them

Note: I can only respond to comments from users who have write access to this repository.

⚙️ Control Options:

  • Disable automatic comment and CI monitoring

@graphite-app graphite-app bot added consumer core area: core, team members only labels Mar 11, 2026
@graphite-app graphite-app bot requested a review from a team March 11, 2026 20:38
Copy link
Contributor

@cubic-dev-ai cubic-dev-ai bot left a comment

Choose a reason for hiding this comment

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

No issues found across 2 files

@paragon-review
Copy link

Paragon: tests updated

1 new test generated for this PR.

New Tests

  • no-plan-files.test

Accept Changes Open in Paragon

Details

New Tests

  • no-plan-files.test (unit)

@Ryukemeister Ryukemeister added the run-ci Approve CI to run for external contributors label Mar 12, 2026
@Ryukemeister Ryukemeister enabled auto-merge (squash) March 12, 2026 08:52
@Ryukemeister Ryukemeister merged commit 26e225e into main Mar 12, 2026
63 of 64 checks passed
@Ryukemeister Ryukemeister deleted the devin/1773261428-remove-plan-files branch March 12, 2026 08:54
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

consumer core area: core, team members only ready-for-e2e run-ci Approve CI to run for external contributors size/L

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants