Skip to content

chore: Suggest npm ci instead of npm install#2928

Open
hmacr wants to merge 1 commit intomainfrom
ser-1176
Open

chore: Suggest npm ci instead of npm install#2928
hmacr wants to merge 1 commit intomainfrom
ser-1176

Conversation

@hmacr
Copy link
Contributor

@hmacr hmacr commented Mar 24, 2026

Towards SER-1176

Summary by CodeRabbit

  • Style
    • Updated placeholder text for installation command input fields across project function and site deployment interfaces.

@appwrite
Copy link

appwrite bot commented Mar 24, 2026

Console (appwrite/console)

Project ID: 688b7bf400350cbd60e9

Sites (1)
Site Status Logs Preview QR
 console-stage
688b7cf6003b1842c9dc
Ready Ready View Logs Preview URL QR Code

Tip

Deploy functions via zip upload or connect directly to your Git repo

@coderabbitai
Copy link
Contributor

coderabbitai bot commented Mar 24, 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: df4074a7-e1ba-4653-a08d-b707b4a9005c

📥 Commits

Reviewing files that changed from the base of the PR and between 5251384 and ceb71a7.

📒 Files selected for processing (5)
  • src/routes/(console)/project-[region]-[project]/functions/create-function/deploy/+page.svelte
  • src/routes/(console)/project-[region]-[project]/functions/create-function/manual/configuration.svelte
  • src/routes/(console)/project-[region]-[project]/functions/create-function/repository-[repository]/configuration.svelte
  • src/routes/(console)/project-[region]-[project]/functions/function-[function]/settings/updateBuildCommand.svelte
  • src/routes/(console)/project-[region]-[project]/sites/create-site/deploy/+page.svelte

Walkthrough

This PR updates the placeholder text for "Install command" input fields across five Svelte component pages. The placeholder is changed from npm install to npm ci in function creation, function settings, and site creation pages. No binding logic, submission behavior, or deployment logic is modified.

Estimated code review effort

🎯 1 (Trivial) | ⏱️ ~3 minutes

🚥 Pre-merge checks | ✅ 3
✅ Passed checks (3 passed)
Check name Status Explanation
Description Check ✅ Passed Check skipped - CodeRabbit’s high-level summary is enabled.
Title check ✅ Passed The title clearly and concisely describes the main change: updating placeholder text suggestions from 'npm install' to 'npm ci' across multiple UI forms.
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 unit tests (beta)
  • Create PR with unit tests
  • Commit unit tests in branch ser-1176

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.

@greptile-apps
Copy link
Contributor

greptile-apps bot commented Mar 24, 2026

Greptile Summary

This PR updates placeholder text across 5 Svelte components to suggest npm ci instead of npm install for install/build command input fields in the Functions and Sites deployment flows.

  • Updated placeholder text in deploy/+page.svelte (functions), manual/configuration.svelte, and repository-[repository]/configuration.svelte from "npm install""npm ci"
  • Updated placeholder in updateBuildCommand.svelte (function settings) from "npm install""npm ci"
  • Updated the fallback placeholder expression in sites/create-site/deploy/+page.svelte from installCommand || 'npm install'installCommand || 'npm ci'
  • Other occurrences of npm install in the codebase (e.g., onboarding SDK installation instructions in createWeb.svelte and createReactNative.svelte) are intentionally left unchanged as they represent a different context (local development setup, not CI/CD pipelines)

Confidence Score: 5/5

  • This PR is safe to merge — all changes are pure placeholder text updates with no logic changes.
  • The changes are minimal and purely cosmetic (placeholder strings only), touching no logic, state, or data flow. All 5 affected files are updated consistently, and the remaining npm install occurrences in the codebase are in different contexts (onboarding instructions) where npm install remains appropriate.
  • No files require special attention.

Important Files Changed

Filename Overview
src/routes/(console)/project-[region]-[project]/functions/create-function/deploy/+page.svelte Placeholder for install command field updated from "e.g., npm install" to "e.g., npm ci"
src/routes/(console)/project-[region]-[project]/functions/create-function/manual/configuration.svelte Placeholder for install command field updated from "npm install" to "npm ci"
src/routes/(console)/project-[region]-[project]/functions/create-function/repository-[repository]/configuration.svelte Placeholder for install command field updated from "npm install" to "npm ci"
src/routes/(console)/project-[region]-[project]/functions/function-[function]/settings/updateBuildCommand.svelte Placeholder for build command field updated from "npm install" to "npm ci"
src/routes/(console)/project-[region]-[project]/sites/create-site/deploy/+page.svelte Fallback placeholder for install command field updated from 'npm install' to 'npm ci'

Reviews (1): Last reviewed commit: "chore: Suggest `npm ci` instead of `npm ..." | Re-trigger Greptile

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