Skip to content

[docs] Note deprecation of project upgrade-check in Docker upgrade guide - #2492

Merged
Micha Hobert (Isengo1989) merged 1 commit into
mainfrom
docs/deprecate-upgrade-check-202d9dcc3f1f9a4e
Sep 8, 2026
Merged

[docs] Note deprecation of project upgrade-check in Docker upgrade guide#2492
Micha Hobert (Isengo1989) merged 1 commit into
mainfrom
docs/deprecate-upgrade-check-202d9dcc3f1f9a4e

Conversation

@shopware-docs-updater

Copy link
Copy Markdown
Contributor

Related

shopware/shopware-cli#1477

What changed?

shopware/shopware-cli#1477 adds a Cobra deprecation notice to shopware-cli project upgrade-check (to be removed in October 2026), replacing it with the guided shopware-cli project upgrade flow.

The Docker upgrade guide still instructs readers to run shopware-cli project upgrade-check as the first step of a manual PHP/Shopware upgrade. This PR adds a note there pointing to the deprecation and the replacement command, so readers are not sent to a command scheduled for removal without guidance.

Files changed

  • guides/hosting/installation-updates/docker.md

cc @shyim (author of the source PR)

Generated by 📝 PR documentation check for #1477 · auto · 22 AIC · ⌖ 9.14 AIC · ⊞ 7.6K ·

Co-authored-by: Copilot <223556219+Copilot@users.noreply.github.com>
@shopware-dev-docs-connector

shopware-dev-docs-connector Bot commented Sep 1, 2026

Copy link
Copy Markdown

Developer Docs healthcheck

Status: Completed with success.
Repository: shopware/docs
Commit: 63f9c6d
Preview: https://developer-documentation-irqr9d3gt-shopware-frontends.vercel.app
Workflow run: #5114

@shyim
Soner (shyim) marked this pull request as ready for review September 8, 2026 11:34
Copilot AI lite review requested due to automatic review settings September 8, 2026 11:34
@github-actions

github-actions Bot commented Sep 8, 2026

Copy link
Copy Markdown
Contributor

📢 Developer Announcement Recommendation

Recommendation: ❌ No Announcement Recommended

No developer announcement signals were detected.

This PR does not appear to introduce developer-facing changes that require a community announcement.

Copilot AI left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

🟡 Changes recommended

The updated step still instructs running a deprecated command, which creates conflicting guidance and should be revised to recommend the replacement workflow directly.

Once you've addressed the issues Copilot identified, you can request another Copilot review.

Pull request overview

Updates the Docker-based Shopware upgrade guide to address the deprecation of shopware-cli project upgrade-check, so readers are directed toward the supported replacement workflow.

Changes:

  • Adds a deprecation/removal note for shopware-cli project upgrade-check in the “major upgrade” step list.
  • Points readers toward using shopware-cli project upgrade as the replacement command.
File summaries
File Description
guides/hosting/installation-updates/docker.md Adds a deprecation note for the legacy upgrade-check command in the Docker upgrade procedure.
Review details
  • Files reviewed: 1/1 changed files
  • Comments generated: 1
  • Review effort level: Lite

💡 Add a code-review agent skill or configure MCP servers for context-aware, tailored reviews. Learn more in the docs.

Comment thread guides/hosting/installation-updates/docker.md

@Isengo1989 Micha Hobert (Isengo1989) left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

Thx for the PR 👍 lets hope it catches the removal in Oktober, I would have removed it already if there is a replacement

@Isengo1989
Micha Hobert (Isengo1989) merged commit 3b4c937 into main Sep 8, 2026
11 checks passed
@Isengo1989
Micha Hobert (Isengo1989) deleted the docs/deprecate-upgrade-check-202d9dcc3f1f9a4e branch September 8, 2026 15:21
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.

3 participants