Skip to content

chore(internal): remove unused Dockerfile.base and cache_heavy_deps.sh#13879

Merged
fern-support merged 1 commit intomainfrom
devin/1774120572-remove-unused-dockerfile-base
Mar 21, 2026
Merged

chore(internal): remove unused Dockerfile.base and cache_heavy_deps.sh#13879
fern-support merged 1 commit intomainfrom
devin/1774120572-remove-unused-dockerfile-base

Conversation

@Swimburger
Copy link
Member

@Swimburger Swimburger commented Mar 21, 2026

Description

Removes two unused files from packages/generators/docker/ that are no longer referenced anywhere in the codebase.

Changes Made

  • Deleted packages/generators/docker/Dockerfile.base — a legacy base image definition (Node 20.18, old fern/hey@buildwithfern.com git identity)
  • Deleted packages/generators/docker/cache_heavy_deps.sh — a yarn cache-warming script only referenced by Dockerfile.base

No other file in the repository references either of these files (verified via repo-wide grep).

Testing

  • Manual verification: grep -r 'Dockerfile.base\|cache_heavy_deps' . returns no results outside the deleted files themselves
  • No unit tests applicable — file deletion only

Human Review Checklist

  • Confirm no CI workflow, Makefile, or external build system references Dockerfile.base or cache_heavy_deps.sh (grep only covers in-repo references)
  • Confirm no other Docker images use FROM fernapi/...-base or similar that could have been built from this Dockerfile

Link to Devin session: https://app.devin.ai/sessions/371cc6bdbad447bb9161b29b0ab8ec28
Requested by: @Swimburger


Open with Devin

Co-Authored-By: Niels Swimberghe <3382717+Swimburger@users.noreply.github.com>
Copy link

@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 skipped — your organization's overage spend limit has been reached.

Code review is billed via overage credits. To resume reviews, an organization admin can raise the monthly limit in Settings → Usage.

Once credits are available, reopen this pull request to trigger a review.

@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. Add '(aside)' to your comment to have me ignore it.
  • 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

Copy link
Contributor

@devin-ai-integration devin-ai-integration bot left a comment

Choose a reason for hiding this comment

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

✅ Devin Review: No Issues Found

Devin Review analyzed this PR and found no bugs or issues to report.

Open in Devin Review

@fern-support fern-support merged commit d31bef1 into main Mar 21, 2026
65 of 66 checks passed
@fern-support fern-support deleted the devin/1774120572-remove-unused-dockerfile-base branch March 21, 2026 21:18
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Development

Successfully merging this pull request may close these issues.

2 participants