Skip to content

feat: update version detection to support ddev head version #433

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Open
wants to merge 1 commit into
base: main
Choose a base branch
from

Conversation

AkibaAT
Copy link
Collaborator

@AkibaAT AkibaAT commented May 1, 2025

The Problem/Issue/Bug:

Currently, ddev head versions that report a version like ddev version v1.24.4-36-ge74e3a95f are not supported by the plugin.

How this PR Solves the Problem:

Updated the version parsing to handle head versions.

Manual Testing Instructions:

Download a DDEV head version: https://ddev.readthedocs.io/en/stable/developers/building-contributing/#testing-latest-commits-on-head
Change the DDEV settings in PhpStorm to use the head version path (/usr/local/bin/ddev)
Check if the plugin still works

Related Issue Link(s):

@AkibaAT AkibaAT requested a review from wazum May 1, 2025 20:45
@AkibaAT AkibaAT self-assigned this May 1, 2025
@AkibaAT AkibaAT added the enhancement New feature or request label May 1, 2025
@AkibaAT AkibaAT force-pushed the feature/support-ddev-head-version branch from 5b1e6ae to af118ed Compare May 1, 2025 20:54
@AkibaAT AkibaAT force-pushed the feature/support-ddev-head-version branch from af118ed to 10cadfe Compare May 1, 2025 21:19
Copy link

sonarqubecloud bot commented May 1, 2025

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
enhancement New feature or request
Projects
None yet
Development

Successfully merging this pull request may close these issues.

1 participant