Skip to content

Remove unused tmp.ts file#1337

Merged
joshunrau merged 4 commits intoDouglasNeuroInformatics:mainfrom
david-roper:remove-tmp-file
Apr 23, 2026
Merged

Remove unused tmp.ts file#1337
joshunrau merged 4 commits intoDouglasNeuroInformatics:mainfrom
david-roper:remove-tmp-file

Conversation

@david-roper
Copy link
Copy Markdown
Collaborator

@david-roper david-roper commented Apr 21, 2026

remove unused tmp.ts file from upload utils

double checked if it was referenced anywhere with claude code.

Summary by CodeRabbit

  • Chores
    • Removed CSV template generation and data upload processing functionality.

@david-roper david-roper requested a review from joshunrau as a code owner April 21, 2026 20:20
@coderabbitai
Copy link
Copy Markdown
Contributor

coderabbitai Bot commented Apr 21, 2026

No actionable comments were generated in the recent review. 🎉

ℹ️ Recent review info
⚙️ Run configuration

Configuration used: Repository UI

Review profile: CHILL

Plan: Pro

Run ID: 178ebf9d-1987-496b-8d1c-21d5d04942c2

📥 Commits

Reviewing files that changed from the base of the PR and between f81db13 and 335997f.

📒 Files selected for processing (1)
  • apps/web/src/utils/tmp.ts
💤 Files with no reviewable changes (1)
  • apps/web/src/utils/tmp.ts

Walkthrough

The PR removes apps/web/src/utils/tmp.ts, which contained a Zod v4-driven namespace for CSV template generation and upload processing with schema parsing, sample data generation, and typed data conversion.

Changes

Cohort / File(s) Summary
File Removal
apps/web/src/utils/tmp.ts
Removed entire module containing Zod4 namespace with CSV template generation, sample data creation, and CSV upload processing logic (437 lines deleted).

Estimated code review effort

🎯 2 (Simple) | ⏱️ ~8 minutes

Possibly related PRs

  • Zod4 upload #1190: This PR directly undoes the additions from #1190 that introduced Zod4-based CSV template generation and processing in the same module.

Suggested reviewers

  • joshunrau
🚥 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 pull request title accurately and clearly summarizes the main change: removing an unused tmp.ts file from the codebase.
Docstring Coverage ✅ Passed No functions found in the changed files to evaluate docstring coverage. Skipping docstring coverage check.
Linked Issues check ✅ Passed Check skipped because no linked issues were found for this pull request.
Out of Scope Changes check ✅ Passed Check skipped because no linked issues were found for this pull request.

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

✨ Finishing Touches
🧪 Generate unit tests (beta)
  • Create PR with unit tests

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.

@joshunrau joshunrau merged commit 3520e24 into DouglasNeuroInformatics:main Apr 23, 2026
2 checks passed
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