Skip to content

Fix: Add missing id arguments to cache mount directives - #1596

Merged
BigSimmo merged 1 commit into
mainfrom
railway/code-change-3-DN-U
Aug 3, 2026
Merged

Fix: Add missing id arguments to cache mount directives#1596
BigSimmo merged 1 commit into
mainfrom
railway/code-change-3-DN-U

Conversation

@railway-app

Copy link
Copy Markdown
Contributor

Problem

Dockerfile.worker failed build validation because the three --mount=type=cache declarations (npm cache at lines 30 and 46, pip cache at line 60) omitted the required id argument, which buildkit needs to identify each cache volume.

Solution

Added id=npm to both npm cache mounts and id=pip to the pip cache mount, matching the required --mount=type=cache,id=<name>,target=<path> syntax so the worker image builds successfully.

Changes

  • ModifiedDockerfile.worker

Generated by Railway

@railway-app
railway-appBot requested a review from BigSimmo as a code ownerAugust 3, 2026 17:51
@supabase

supabaseBot commented Aug 3, 2026

Copy link
Copy Markdown

This pull request has been ignored for the connected project sjrfecxgysukkwxsowpy because there are no changes detected in supabase directory. You can change this behaviour in Project Integrations Settings ↗︎.


Preview Branches by Supabase.
Learn more about Supabase Branching ↗︎.

@coderabbitai

Copy link
Copy Markdown
Contributor

Important

Review skipped

Bot user detected.

To trigger a single review, invoke the @coderabbitai review command.

⚙️ Run configuration

Configuration used: Path: .coderabbit.yaml

Review profile: CHILL

Plan: Free

Run ID: cface56e-9377-4673-92ae-25e95ea35a1c

You can disable this status message by setting the reviews.review_status to false in the CodeRabbit configuration file.

Use the checkbox below for a quick retry:

  • 🔍 Trigger review

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

@BigSimmo
BigSimmo merged commit 838c170 into mainAug 3, 2026
28 checks passed
@BigSimmo
BigSimmo deleted the railway/code-change-3-DN-U branch August 3, 2026 17:59
Sign up for freeto 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

@BigSimmo