Update Meilisearch 1.35.1 and fix Cyberchef logo#722
Update Meilisearch 1.35.1 and fix Cyberchef logo#722kadma wants to merge 2 commits intoDokploy:canaryfrom
Conversation
built with Refined Cloudflare Pages Action⚡ Cloudflare Pages Deployment
|
| services: | ||
| meilisearch: | ||
| image: getmeili/meilisearch:v1.8.3 | ||
| image: getmeili/meilisearch:v1.35.1 |
There was a problem hiding this comment.
Missing restart policy. According to AGENTS.md guidelines, all services MUST include restart: unless-stopped or restart: always.
| image: getmeili/meilisearch:v1.35.1 | |
| image: getmeili/meilisearch:v1.35.1 | |
| restart: unless-stopped |
Context Used: Context from dashboard - AGENTS.md (source)
| [config.env] | ||
| MEILI_ENV = "development" | ||
| MEILI_MASTER_KEY = "${master_key}" | ||
| MEILI_MASTER_KEY = "${master_key}" No newline at end of file |
There was a problem hiding this comment.
File is missing trailing newline at end of file.
Note: If this suggestion doesn't match your team's coding style, reply to this and let me know. I'll remember it for next time!
There was a problem hiding this comment.
Pull request overview
This PR updates the Meilisearch template to version v1.35.1 (from v1.8.3) and fixes the CyberChef logo reference to use a PNG file instead of SVG. The Meilisearch update includes removing the MEILI_ENV environment variable, which aligns with how other blueprints in the repository configure Meilisearch (none of them use MEILI_ENV).
Changes:
- Updated Meilisearch from v1.8.3 to v1.35.1 in both meta.json and docker-compose.yml
- Removed MEILI_ENV environment variable from Meilisearch template (consistent with other blueprints)
- Changed CyberChef logo from cyberchef.svg to cyberchef.png in meta.json and replaced the logo file
Reviewed changes
Copilot reviewed 3 out of 5 changed files in this pull request and generated 1 comment.
Show a summary per file
| File | Description |
|---|---|
| meta.json | Updated Meilisearch version to v1.35.1 and changed CyberChef logo reference from .svg to .png |
| blueprints/meilisearch/docker-compose.yml | Updated Meilisearch Docker image from v1.8.3 to v1.35.1 and removed MEILI_ENV environment variable |
| blueprints/meilisearch/template.toml | Removed MEILI_ENV configuration, keeping only MEILI_MASTER_KEY |
| blueprints/cyberchef/cyberchef.svg | Deleted old SVG logo file |
| blueprints/cyberchef/cyberchef.png | Added new PNG logo file |
| services: | ||
| meilisearch: | ||
| image: getmeili/meilisearch:v1.8.3 | ||
| image: getmeili/meilisearch:v1.35.1 |
There was a problem hiding this comment.
The version jump from v1.8.3 to v1.35.1 is substantial (27 minor versions). According to project conventions, Docker images must be verified to exist using docker manifest inspect getmeili/meilisearch:v1.35.1 before committing to ensure the image is available and prevent deployment failures. While the PR checklist indicates the template has been tested, please confirm this specific version exists on Docker Hub.
What is this PR about?
New PR of meilisearch and cyberchef
Checklist
Before submitting this PR, please make sure that:
Greptile Summary
Updated Meilisearch from v1.8.3 to v1.35.1 and replaced CyberChef logo from SVG to PNG format.
docker-compose.ymlandmeta.jsonMEILI_ENVenvironment variable from Meilisearch configurationmeta.jsonConfidence Score: 4/5
restartpolicy per project guidelines, which is a critical requirement for production deployments. Once this is fixed, the PR will be safe to merge.blueprints/meilisearch/docker-compose.ymlto ensure restart policy is addedLast reviewed commit: e78305f
Context used:
dashboard- AGENTS.md (source)