Skip to content

fix: update instructions and enhance command clarity in setup steps#3849

Merged
rohitpaulk merged 1 commit into
mainfrom
andy/tweak-setup-and-stage-1-text
Apr 14, 2026
Merged

fix: update instructions and enhance command clarity in setup steps#3849
rohitpaulk merged 1 commit into
mainfrom
andy/tweak-setup-and-stage-1-text

Conversation

@andy1li

@andy1li andy1li commented Apr 13, 2026

Copy link
Copy Markdown
Member
  • Corrected wording in the task card to refer to a single step.
  • Added comments to command variants in the CLI connection setup for better understanding.
image image

Checklist:

  • I've thoroughly self-reviewed my changes
  • I've added tests for my changes, unless they affect admin-only areas (or are otherwise not worth testing)
  • I've verified any visual changes using Percy (add a commit with [percy] in the message to trigger)

Note

Low Risk
Low risk copy-only changes to course/setup UI text and displayed terminal commands with no behavioral logic changes.

Overview
Updates the first-stage task card copy to reference a single step instead of multiple steps.

Annotates the displayed CLI install commands for Linux/macOS and PowerShell with inline comments ("Install CodeCrafters CLI") while keeping the subsequent codecrafters ping command unchanged.

Reviewed by Cursor Bugbot for commit 7c43581. Bugbot is set up for automated code reviews on this repo. Configure here.

- Corrected wording in the task card to refer to a single step.
- Added comments to command variants in the CLI connection setup for better understanding.
@andy1li andy1li self-assigned this Apr 13, 2026
@codecov

codecov Bot commented Apr 13, 2026

Copy link
Copy Markdown

Bundle Report

Changes will decrease total bundle size by 1.59kB (-0.0%) ⬇️. This is within the configured threshold ✅

Detailed changes
Bundle name Size Change
client-array-push 39.47MB -1.59kB (-0.0%) ⬇️

Affected Assets, Files, and Routes:

view changes for bundle: client-array-push

Assets Changed:

Asset Name Size Change Total Size Change (%)
assets/chunk.*.js 53 bytes 388.56kB 0.01%
assets/chunk.*.js -82 bytes 41.53kB -0.2%
assets/chunk.*.js -1.56kB 3.43MB -0.05%

@github-actions

github-actions Bot commented Apr 13, 2026

Copy link
Copy Markdown

Test Results

  1 files  ±0    1 suites  ±0   8m 35s ⏱️ - 3m 1s
707 tests ±0  644 ✅ +2  63 💤 ±0  0 ❌ ±0 
722 runs  ±0  659 ✅ +4  63 💤 ±0  0 ❌  - 2 

Results for commit 7c43581. ± Comparison against base commit c167f5e.

♻️ This comment has been updated with latest results.

@codecov

codecov Bot commented Apr 13, 2026

Copy link
Copy Markdown

Codecov Report

✅ All modified and coverable lines are covered by tests.
✅ All tests successful. No failed tests found.

📢 Thoughts on this report? Let us know!

@andy1li andy1li requested a review from rohitpaulk April 13, 2026 20:45
@rohitpaulk rohitpaulk merged commit e002b59 into main Apr 14, 2026
9 of 10 checks passed
@rohitpaulk rohitpaulk deleted the andy/tweak-setup-and-stage-1-text branch April 14, 2026 11:01
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