Skip to content

Phpmyadmin isLegacy duplication#79

Closed
cursor[bot] wants to merge 11 commits into
mainfrom
cursor/phpmyadmin-islegacy-duplication-ab9f
Closed

Phpmyadmin isLegacy duplication#79
cursor[bot] wants to merge 11 commits into
mainfrom
cursor/phpmyadmin-islegacy-duplication-ab9f

Conversation

@cursor

@cursor cursor Bot commented Feb 22, 2026

Copy link
Copy Markdown

Bare minimum self-checks

  • I've updated this PR with the latest code from main
  • I've done a cursory QA pass of my code locally
  • I've ensured all automated status check and tests pass
  • I've connected this PR to an issue

Pieces of flare

  • I've written a unit or functional test for my code
  • I've updated relevant documentation it my code changes it
  • I've updated this repo's README if my code changes it
  • I've updated this repo's CHANGELOG with my change unless its a trivial change (like updating a typo in the docs)

Finally

If you have any issues or need help please join the #contributors channel in the Lando slack and someone will gladly help you out!

You can also check out the coder guide.


Refactors phpmyadmin.js to define the isLegacy version check once and reuse it for both command and image selection. This eliminates duplicated semver comparisons, reducing maintenance risk and improving code quality.


AaronFeledy and others added 11 commits February 22, 2026 14:41
The official phpmyadmin Docker image only has 5.x tags. Legacy 4.x
versions (4.9, 4.8, 4.7, 4.6) are only available on the deprecated
phpmyadmin/phpmyadmin image. This change uses the appropriate image
based on version to prevent Docker pull failures for legacy versions.

Applied via @cursor push command
Change version threshold from 5.0.0 to 5.2.0 to ensure versions 5.0 and 5.1 use the deprecated phpmyadmin/phpmyadmin image where these tags exist, since the official phpmyadmin image only supports 5.2.x and higher.

Applied via @cursor push command
@cursor

cursor Bot commented Feb 22, 2026

Copy link
Copy Markdown
Author

Cursor Agent can help with this pull request. Just @cursor in comments and I'll start working on changes in this branch.
Learn more about Cursor Agents

@cursor

cursor Bot commented Feb 23, 2026

Copy link
Copy Markdown
Author

Automatically closed this PR because it was created by Bugbot autofix for #74, and that PR was closed.

@cursor cursor Bot closed this Feb 23, 2026
Base automatically changed from fix/official-docker-image to main February 23, 2026 00:33
@cursor cursor Bot deleted the cursor/phpmyadmin-islegacy-duplication-ab9f branch February 23, 2026 00:33
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.

2 participants