Skip to content

Update Docker images - #9

Merged
adriaanzon merged 2 commits into
masterfrom
update-docker-images
Jul 15, 2026
Merged

Update Docker images#9
adriaanzon merged 2 commits into
masterfrom
update-docker-images

Conversation

@adriaanzon

@adriaanzon adriaanzon commented Jul 13, 2026

Copy link
Copy Markdown
Member

Summary by CodeRabbit

  • Documentation

    • Updated local PrestaShop installation instructions with the current /admin-dev login paths.
  • Chores

    • Updated the PrestaShop 9 development environment to use version 9.1.1.
    • Standardized the administrative folder configuration across supported environments.
    • Removed automatic container restart policies from local development services.
  • Tests

    • Updated static analysis configuration to account for a known nullable context property warning.

Module::$context is nullable PrestaShop 9.1.
@coderabbitai

coderabbitai Bot commented Jul 13, 2026

Copy link
Copy Markdown

Review Change Stack

No actionable comments were generated in the recent review. 🎉

ℹ️ Recent review info
⚙️ Run configuration

Configuration used: defaults

Review profile: CHILL

Plan: Pro Plus

Run ID: 326633d6-24d4-46be-ba2d-924ef99ac67a

📥 Commits

Reviewing files that changed from the base of the PR and between 226836f and c01c41d.

📒 Files selected for processing (6)
  • .github/workflows/analysis.yml
  • CONTRIBUTING.md
  • docker-compose.1.7.yml
  • docker-compose.8.yml
  • docker-compose.9.yml
  • tests/phpstan/phpstan-baseline.9.neon

📝 Walkthrough

Walkthrough

Updated local admin URLs and Compose settings, moved the PrestaShop 9 environment to image 9.1.1, aligned PHPStan analysis with that image, and added one PHPStan baseline ignore.

Changes

PrestaShop environment alignment

Layer / File(s) Summary
Local stack settings
CONTRIBUTING.md, docker-compose.1.7.yml, docker-compose.8.yml, docker-compose.9.yml
Back-office URLs and PS_FOLDER_ADMIN use admin-dev; restart policies are removed from MySQL and PrestaShop services.
PrestaShop 9 analysis alignment
.github/workflows/analysis.yml, docker-compose.9.yml, tests/phpstan/phpstan-baseline.9.neon
PrestaShop 9 uses image tag 9.1.1 in Compose and PHPStan, with one nullable Context property diagnostic added to the baseline.

Estimated code review effort: 2 (Simple) | ~10 minutes

🚥 Pre-merge checks | ✅ 5
✅ Passed checks (5 passed)
Check name Status Explanation
Description Check ✅ Passed Check skipped - CodeRabbit’s high-level summary is enabled.
Title check ✅ Passed The title matches the main theme of the changeset, which updates Docker image versions and related container configuration.
Docstring Coverage ✅ Passed No functions found in the changed files to evaluate docstring coverage. Skipping docstring coverage check.
Linked Issues check ✅ Passed Check skipped because no linked issues were found for this pull request.
Out of Scope Changes check ✅ Passed Check skipped because no linked issues were found for this pull request.
✨ Finishing Touches
🧪 Generate unit tests (beta)
  • Create PR with unit tests
  • Commit unit tests in branch update-docker-images

Warning

Tools execution failed with the following error:

Failed to run tools: 13 INTERNAL: Received RST_STREAM with code 2 (Internal server error)


Comment @coderabbitai help to get the list of available commands.

@adriaanzon
adriaanzon merged commit 9d5bd53 into master Jul 15, 2026
9 checks passed
@adriaanzon
adriaanzon deleted the update-docker-images branch July 15, 2026 12:06
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.

1 participant