Skip to content

chore: Remove unused files (#3286)#3287

Merged
KATO-Hiro merged 1 commit into
stagingfrom
#3286
Mar 20, 2026
Merged

chore: Remove unused files (#3286)#3287
KATO-Hiro merged 1 commit into
stagingfrom
#3286

Conversation

@KATO-Hiro
Copy link
Copy Markdown
Collaborator

@KATO-Hiro KATO-Hiro commented Mar 20, 2026

close #3286

Summary by CodeRabbit

  • Chores
    • Removed development infrastructure and internal configuration files including test automation scripts, test environment setup data, automated dependency management settings, and legacy database migration components to streamline repository organization and reduce maintenance overhead.

@coderabbitai
Copy link
Copy Markdown
Contributor

coderabbitai Bot commented Mar 20, 2026

No actionable comments were generated in the recent review. 🎉

ℹ️ Recent review info
⚙️ Run configuration

Configuration used: Organization UI

Review profile: CHILL

Plan: Pro

Run ID: c8421d51-ea79-4a5d-9784-a98d83dccd4c

📥 Commits

Reviewing files that changed from the base of the PR and between 372d78a and eef93a3.

📒 Files selected for processing (5)
  • prisma/users_for_test.ts
  • renovate.json
  • supabase/migrations/20231122085404_remote_schema.sql
  • tests/global.setup.ts
  • tests/global.teardown.ts
💤 Files with no reviewable changes (5)
  • renovate.json
  • prisma/users_for_test.ts
  • tests/global.teardown.ts
  • tests/global.setup.ts
  • supabase/migrations/20231122085404_remote_schema.sql

Walkthrough

This pull request removes five unused files and configurations: a test user fixture module, Renovate automation configuration, a Supabase SQL migration, and test setup/teardown scripts. The deletion eliminates 273 lines of code without adding new functionality, supporting code cleanup efforts.

Changes

Cohort / File(s) Summary
Configuration Files
renovate.json
Removed entire Renovate configuration including schema reference, scheduling rules, and dashboard settings.
Test Fixtures & Setup
prisma/users_for_test.ts, tests/global.setup.ts, tests/global.teardown.ts
Removed test user constant array and associated Prisma-backed setup/teardown scripts that registered and cleaned up test user records.
Database Migrations
supabase/migrations/20231122085404_remote_schema.sql
Removed SQL migration containing PostgreSQL extensions initialization, schema setup, ENUM type definition (public.Roles), and core table definitions (public.key, public.session, public.user) with privileges and constraints.

Estimated code review effort

🎯 2 (Simple) | ⏱️ ~10 minutes

Poem

🐰 Files deleted, clutter cleared,
Unused code has disappeared,
Migrations gone, configs too,
A cleaner repo, fresh and new! ✨

🚥 Pre-merge checks | ✅ 5
✅ Passed checks (5 passed)
Check name Status Explanation
Description Check ✅ Passed Check skipped - CodeRabbit’s high-level summary is enabled.
Title check ✅ Passed The title 'chore: Remove unused files' clearly and concisely describes the main change in the pull request, which removes five unused files from the codebase.
Linked Issues check ✅ Passed The PR closes issue #3286 which requests removal of unused files and code. The changes remove five files deemed unused: prisma/users_for_test.ts, renovate.json, a migration file, and test setup/teardown files.
Out of Scope Changes check ✅ Passed All changes in the PR are directly related to removing unused files as specified in the linked issue #3286. No unrelated modifications are present.
Docstring Coverage ✅ Passed No functions found in the changed files to evaluate docstring coverage. Skipping docstring coverage check.

✏️ Tip: You can configure your own custom pre-merge checks in the settings.

✨ Finishing Touches
📝 Generate docstrings
  • Create stacked PR
  • Commit on current branch
🧪 Generate unit tests (beta)
  • Create PR with unit tests
  • Commit unit tests in branch #3286
📝 Coding Plan
  • Generate coding plan for human review comments

Thanks for using CodeRabbit! It's free for OSS, and your support helps us grow. If you like it, consider giving us a shout-out.

❤️ Share

Comment @coderabbitai help to get the list of available commands and usage tips.

Copy link
Copy Markdown
Collaborator Author

@KATO-Hiro KATO-Hiro left a comment

Choose a reason for hiding this comment

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

LGTM

@KATO-Hiro KATO-Hiro merged commit c85af54 into staging Mar 20, 2026
3 checks passed
@KATO-Hiro KATO-Hiro deleted the #3286 branch March 20, 2026 01:35
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.

使用していないファイル・コードを削除しましょう

1 participant