Skip to content

Bump PHP image version#239

Merged
PGijsbers merged 1 commit intomainfrom
update/php-api-version
Feb 15, 2026
Merged

Bump PHP image version#239
PGijsbers merged 1 commit intomainfrom
update/php-api-version

Conversation

@PGijsbers
Copy link
Contributor

Closes #141

@PGijsbers PGijsbers added the maintenance improvements or changes to existing systems label Feb 13, 2026
@coderabbitai
Copy link
Contributor

coderabbitai bot commented Feb 13, 2026

Walkthrough

The pull request updates the docker-compose.yaml configuration file to upgrade the PHP API service to a newer version. Specifically, the image tag for the php-api service is changed from openml/php-rest-api:v1.2.2 to openml/php-rest-api:v1.2.4. This is a single-line configuration modification with no changes to service dependencies, method signatures, or control flow logic.

🚥 Pre-merge checks | ✅ 6
✅ Passed checks (6 passed)
Check name Status Explanation
Title check ✅ Passed The title 'Bump PHP image version' directly and clearly summarizes the main change: updating the PHP API image tag from v1.2.2 to v1.2.4.
Description check ✅ Passed The description references the linked issue #141, which is directly related to updating the PHP image to use the newest PHP API as implemented in the changeset.
Linked Issues check ✅ Passed The PR successfully updates the PHP image from v1.2.2 to v1.2.4 in docker-compose.yaml, directly fulfilling the requirement in issue #141 to use the newest PHP API version.
Out of Scope Changes check ✅ Passed All changes are in-scope: only the PHP API image tag is updated in docker-compose.yaml, directly addressing issue #141 with no extraneous modifications.
Docstring Coverage ✅ Passed No functions found in the changed files to evaluate docstring coverage. Skipping docstring coverage check.
Merge Conflict Detection ✅ Passed ✅ No merge conflicts detected when merging into main

✏️ Tip: You can configure your own custom pre-merge checks in the settings.

✨ Finishing touches
🧪 Generate unit tests (beta)
  • Create PR with unit tests
  • Post copyable unit tests in a comment
  • Commit unit tests in branch update/php-api-version

No actionable comments were generated in the recent review. 🎉


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.

❤️ Share

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

Copy link
Contributor

@sourcery-ai sourcery-ai bot left a comment

Choose a reason for hiding this comment

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

Hey - I've reviewed your changes and they look great!


Sourcery is free for open source - if you like our reviews please consider sharing them ✨
Help me be more useful! Please click 👍 or 👎 on each comment and I'll use the feedback to improve your reviews.

@codecov
Copy link

codecov bot commented Feb 13, 2026

Codecov Report

✅ All modified and coverable lines are covered by tests.
⚠️ Please upload report for BASE (main@294ca47). Learn more about missing BASE report.

Additional details and impacted files
@@           Coverage Diff           @@
##             main     #239   +/-   ##
=======================================
  Coverage        ?   53.58%           
=======================================
  Files           ?       32           
  Lines           ?     1116           
  Branches        ?      101           
=======================================
  Hits            ?      598           
  Misses          ?      516           
  Partials        ?        2           

☔ View full report in Codecov by Sentry.
📢 Have feedback on the report? Share it here.

🚀 New features to boost your workflow:
  • ❄️ Test Analytics: Detect flaky tests, report on failures, and find test suite problems.

@PGijsbers PGijsbers merged commit 7c177b6 into main Feb 15, 2026
9 checks passed
@PGijsbers PGijsbers deleted the update/php-api-version branch February 15, 2026 19:13
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

maintenance improvements or changes to existing systems

Projects

None yet

Development

Successfully merging this pull request may close these issues.

Update PHP image to use newest PHP API

1 participant