Bump Meilisearch to v1.48#2173
Conversation
📝 WalkthroughWalkthroughThe Meilisearch Enterprise image tag is updated to v1.48 in CI and docker-compose, and the pre-release workflow now runs only for ChangesMeilisearch Version Bump
Pre-release workflow gating
Estimated code review effort: 2 (Simple) | ~10 minutes Possibly related PRs
Suggested reviewers: Poem
🚥 Pre-merge checks | ✅ 5✅ Passed checks (5 passed)
✨ Finishing Touches🧪 Generate unit tests (beta)
Thanks for using CodeRabbit! It's free for OSS, and your support helps us grow. If you like it, consider giving us a shout-out. Comment |
There was a problem hiding this comment.
Actionable comments posted: 1
🤖 Prompt for all review comments with AI agents
Verify each finding against the current code and only fix it if needed.
Inline comments:
In `@docker-compose.yml`:
- Line 17: Replace the non-existent enterprise image tag to restore CI/local
alignment: in docker-compose.yml change the image reference
"getmeili/meilisearch-enterprise:v1.42" to the open-source image
"getmeili/meilisearch:latest" (update the image field) so the compose file pulls
a valid tag and matches CI usage.
🪄 Autofix (Beta)
Fix all unresolved CodeRabbit comments on this PR:
- Push a commit to this branch (recommended)
- Create a new PR with the fixes
ℹ️ Review info
⚙️ Run configuration
Configuration used: Organization UI
Review profile: CHILL
Plan: Pro
Run ID: fb203baa-14aa-4315-ad0b-c27d450bb189
📒 Files selected for processing (3)
.github/workflows/tests.ymldocker-compose.ymltests/get_search.test.ts
|
the name of the branch triggers the pre-release workflows, which are failing, i need to fix that |
6d9a84a to
1a2500c
Compare
Codecov Report✅ All modified and coverable lines are covered by tests. Additional details and impacted files@@ Coverage Diff @@
## main #2173 +/- ##
=======================================
Coverage 98.07% 98.07%
=======================================
Files 14 14
Lines 674 674
Branches 109 109
=======================================
Hits 661 661
Misses 12 12
Partials 1 1 ☔ View full report in Codecov by Harness. 🚀 New features to boost your workflow:
|
There was a problem hiding this comment.
Actionable comments posted: 1
🤖 Prompt for all review comments with AI agents
Verify each finding against current code. Fix only still-valid issues, skip the
rest with a brief reason, keep changes minimal, and validate.
Inline comments:
In @.github/workflows/pre-release-tests.yml:
- Around line 8-20: The workflow exclusion is still scoped to only
`pre-release-beta/` and `prototype-beta/`, so it does not match the new
`pre-release-**` trigger behavior and can cause duplicate `integration_tests`
runs. Update the branch exclusion logic in `tests.yml` to cover all
`pre-release-*` branches, or narrow the `meilisearch-version` trigger in
`pre-release-tests.yml` so the two workflows do not overlap. Use the existing
workflow/job conditions around `integration_tests` and `meilisearch-version` to
keep the branch matching consistent.
🪄 Autofix (Beta)
Fix all unresolved CodeRabbit comments on this PR:
- Push a commit to this branch (recommended)
- Create a new PR with the fixes
ℹ️ Review info
⚙️ Run configuration
Configuration used: Organization UI
Review profile: CHILL
Plan: Pro
Run ID: 9a2816b8-3d38-4281-a42f-d015a0c57d66
📒 Files selected for processing (1)
.github/workflows/pre-release-tests.yml
This PR is auto-generated.
Tracks
meilisearch/meilisearchstable releases and updates pinned image tags to floating minorv1.42.Patch releases are intentionally not PR'd; Docker resolves to the latest patch for this minor.
v1.42.1.github/workflows/tests.ymldocker-compose.ymlRelease notes
Summary by CodeRabbit
pre-release-**branches only.