Skip to content

fix: Pull latest repo changes before running setup#2

Merged
scottlovegrove merged 1 commit intomainfrom
bootstrap-pull-latest
Apr 4, 2026
Merged

fix: Pull latest repo changes before running setup#2
scottlovegrove merged 1 commit intomainfrom
bootstrap-pull-latest

Conversation

@scottlovegrove
Copy link
Copy Markdown
Collaborator

Summary

  • Bootstrap scripts (bootstrap-mac.sh and bootstrap-windows.ps1) now git pull --rebase origin main when the doist-os repo is already cloned, before running the setup script.
  • Fixes failures caused by stale setup scripts (e.g. removed outline-cli npm link logic still executing on users with older local checkouts).

Test plan

  • Run curl -fsSL https://raw.githubusercontent.com/Doist/dos-scripts/bootstrap-pull-latest/install/bootstrap-mac.sh | bash on a Mac with an existing ~/doist-os clone
  • Confirm bootstrap pulls latest and setup completes successfully

🤖 Generated with Claude Code

The bootstrap scripts skip git pull when the repo is already cloned,
which can run stale setup scripts and cause failures (e.g. removed
outline-cli npm link logic still executing).

Co-Authored-By: Claude Opus 4.6 (1M context) <noreply@anthropic.com>
@doistbot doistbot requested a review from hackertronix April 4, 2026 20:31
@scottlovegrove scottlovegrove self-assigned this Apr 4, 2026
@scottlovegrove scottlovegrove added the 👀 Show PR PR must be reviewed before or after merging label Apr 4, 2026
@scottlovegrove scottlovegrove merged commit 61df06c into main Apr 4, 2026
1 check passed
@scottlovegrove scottlovegrove deleted the bootstrap-pull-latest branch April 4, 2026 20:31
@doistbot
Copy link
Copy Markdown
Member

doistbot commented Apr 4, 2026

⚠️ Doistbot encountered an error while reviewing this PR.

  • Status: FAILED
  • Delivery: 309c3610-3065-11f1-9480-2bfd51d9aa6d
  • Logs: Datadog

⚠️ Doistbot hit an error while reviewing this PR:

Failed to clone repository Doist/dos-scripts

Please, try the command again or contact the team maintaining the bot.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

👀 Show PR PR must be reviewed before or after merging

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants