Skip to content

Conversation

@shyim
Copy link
Member

@shyim shyim commented Jan 28, 2026

Fixes #2081

@shyim shyim requested a review from sushmangupta January 28, 2026 05:35

### JavaScript

* Node.js
Copy link
Member

Choose a reason for hiding this comment

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

should we mention, that node is only required, if you want to compile the JS on the server?

Copy link
Contributor

Choose a reason for hiding this comment

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

Yes, please. This not a requirement to run it. But as macOS is also listed here and likely also not the platform for a production hosting, you really want to keep it in the list but mark it optional. I assume, that you also want to name shopware-cli in combination with asset compilation

@Isengo1989 Isengo1989 added the Blocked Block PRs from merging label Jan 28, 2026
@Isengo1989
Copy link
Contributor

Ok, there were a lot of changes recently. I wouldn't change all of this back to how it was without a proper revision, tbh, so I added the blocked tag for now. It is an essential part of the docs. Did you get a lot of negative feedback recently or did you experience missing parts?

Believe there is a big gap at the moment - best to find a middle ground on what was adjusted in the early stages of the DX changes by @lasomethingsomething - to avoid back-and-forth.

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

Labels

Blocked Block PRs from merging

Projects

None yet

Development

Successfully merging this pull request may close these issues.

Requirements should just include software requirements

5 participants