Skip to content

Document astro dev --no-lock flag#14212

Open
matthewp wants to merge 2 commits into
mainfrom
dev-no-lock-flag-docs
Open

Document astro dev --no-lock flag#14212
matthewp wants to merge 2 commits into
mainfrom
dev-no-lock-flag-docs

Conversation

@matthewp

@matthewp matthewp commented Jul 9, 2026

Copy link
Copy Markdown
Contributor

Documents the new astro dev --no-lock flag added in withastro/astro#17331.

  • Adds a --no-lock reference section under astro dev
  • Lists --no-lock in the astro dev help output

@astrobot-houston

Copy link
Copy Markdown
Contributor

Lunaria Status Overview

🌕 This pull request will trigger status changes.

Learn more

By default, every PR changing files present in the Lunaria configuration's files property will be considered and trigger status changes accordingly.

You can change this by adding one of the keywords present in the ignoreKeywords property in your Lunaria configuration file in the PR's title (ignoring all files) or by including a tracker directive in the merged commit's description.

Tracked Files

File Note
en/reference/cli-reference.mdx Source changed, localizations will be marked as outdated.
Warnings reference
Icon Description
🔄️ The source for this localization has been updated since the creation of this pull request, make sure all changes in the source have been applied.

@github-actions

github-actions Bot commented Jul 9, 2026

Copy link
Copy Markdown

Preview deployment

✅ Deployment complete!

@yanthomasdev yanthomasdev left a comment

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

LGTM

@ArmandPhilippot ArmandPhilippot left a comment

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

Mostly a nit regarding formatting (single/two paragraphs as the two statements do not seem to be linked), but I also think it's important to mention what happens when these flags are combined (see my suggestion). Otherwise, LGTM!

astro dev --no-lock --port 4322
```

The new server is not tracked by `astro dev stop`, `astro dev status`, or `astro dev logs`. `--no-lock` cannot be combined with `--background` (including when background mode is triggered automatically for an AI coding agent) or `--force`, since both rely on the lock file.

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

Reading that, I wondered what happens when these flags are combined: does one takes precedence? do I get an error? Looking at the JSDoc, it seems this throws an error. I think it's important to mention it.

Also, I think the parentheses can be shortened.

But, I'm not sure how to combine both in a logical way. I mean the two statements are not tied so I'm not sure a single paragraph makes sense... and maybe two independent paragraphs are easier to scan?

Suggested change
The new server is not tracked by `astro dev stop`, `astro dev status`, or `astro dev logs`. `--no-lock` cannot be combined with `--background` (including when background mode is triggered automatically for an AI coding agent) or `--force`, since both rely on the lock file.
The new server is not tracked by `astro dev stop`, `astro dev status`, or `astro dev logs`.
When combined with `--background` (including when triggered by an AI coding agent) or `--force`, an error is throwned, as both rely on the lock file.

@ArmandPhilippot ArmandPhilippot added add new content Document something that is not in docs. May require testing, confirmation, or affect other pages. merge-on-release Don't merge this before the feature is released! (MQ=approved but WAIT for feature release!) minor-release For the next minor release; in the milestone, "merge queue" = approved for merging on release day. labels Jul 9, 2026
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

add new content Document something that is not in docs. May require testing, confirmation, or affect other pages. merge-on-release Don't merge this before the feature is released! (MQ=approved but WAIT for feature release!) minor-release For the next minor release; in the milestone, "merge queue" = approved for merging on release day.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants