Repository navigation
docs: update Devenv legacy setup for frosh/devenv-meta 0.4 - #2558
Merged
Micha Hobert (Isengo1989) merged 3 commits intoOct 8, 2026
Merged
Micha Hobert (Isengo1989) merged 3 commits into
Micha Hobert (Isengo1989) merged 3 commits into
Conversation
Bring the Devenv pages in line with the 0.4 recipe and Devenv 2.4: - Create the project and add frosh/devenv-meta with `nix run` Composer, so no PHP or Composer is needed on the host - Require Devenv 2.4+, install it with `nix profile add nixpkgs#devenv` - Document strict ports, automatic DATABASE_URL/MAILER_DSN and the automatically created database and user - MailHog → Mailpit, Adminer port 9080 → 8010, mention Caddy's admin API - Fix demo data instructions (needs shopware/dev-tools; `composer setup` only exists in shopware/shopware) - Add RabbitMQ, OpenSearch and "multiple projects" sections - Use lib.mkForce where the recipe sets values explicitly (MySQL package, Caddy virtual hosts), and update the sales channel domain on port changes - Replace the broken mysql80 overrideAttrs example with a pinned nixpkgs input - Replace non-existent `devenv reload` and the kill-all-Nix-processes command - Fix indentation of all Nix examples and remove nodejs-18_x Co-Authored-By: Claude Opus 5.5 <noreply@anthropic.com>
Developer Docs healthcheckStatus: Completed with Fork PRs need approval from a maintainer with write access before the healthcheck can run. GitHub approves only the latest commit, so every new commit pushed after approval needs another approval. |
Contributor
|
Glad to see updates on a "legacy" setup :) |
Micha Hobert (Isengo1989)
added
the
Improvement
Micha Hobert (Isengo1989)
self-requested a review
October 5, 2026 09:35
Contributor
There was a problem hiding this comment.
Copilot review overview
🟡 Changes recommended
The multi-project Caddy guidance and custom-domain database command can produce incorrect configurations.
Review effort: Balanced
Findings: 4
Open (4)
What changed in this PR
Updates the legacy Shopware Devenv documentation for frosh/devenv-meta 0.4 and Devenv 2.4.
Changes:
- Updates installation, services, ports, and environment variables.
- Adds RabbitMQ, OpenSearch, and multi-project guidance.
- Adds
nixpkgsto the spellcheck wordlist.
| File | Description |
|---|---|
.wordlist.txt |
Allows the nixpkgs term |
devenv-setup.md |
Updates setup and service documentation |
devenv-options.md |
Expands customization guidance |
💡 Add a code-review agent skill or configure MCP servers for context-aware, tailored reviews. Learn more in the docs.
The SQL example hardcoded http://127.0.0.1:8001, which only matched the "Change port only" tab. Use the APP_URL and MYSQL_TCP_PORT variables that Devenv exports instead, so the command matches every example on the page. Co-Authored-By: Claude Opus 5.5 <noreply@anthropic.com>
Caddy opens all TCP listeners, including the admin API, with SO_REUSEPORT on Unix systems, so multiple Devenv projects share 127.0.0.1:2019. Co-Authored-By: Claude Opus 5.5 <noreply@anthropic.com>
Micha Hobert (Isengo1989)
approved these changes
Oct 8, 2026
Contributor
|
Thank you for the PR Gregor Lohaus (@GregorLohaus) 👍 |
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
Summary
Bring the Devenv pages in line with the 0.4 recipe and Devenv 2.4
Related links
pull request
Checklist
PageRefreferences where relevant..gitbook.yamlif pages were moved, renamed, or deleted..wordlist.txt(and sorted it) if spellcheck flags new legitimate terms.Notes
nixpkgsto.wordlist.txt.