Skip to content

fix: show correct project name in ddev mutagen st, fixes #7969#7970

Merged
rfay merged 2 commits intoddev:mainfrom
stasadev:20251222_stasadev_mutagen_st_name
Dec 23, 2025
Merged

fix: show correct project name in ddev mutagen st, fixes #7969#7970
rfay merged 2 commits intoddev:mainfrom
stasadev:20251222_stasadev_mutagen_st_name

Conversation

@stasadev
Copy link
Copy Markdown
Member

The Issue

How This PR Solves The Issue

Fixes it.

Manual Testing Instructions

ddev config --performance-mode=mutagen --project-name=shopware6.7.5.1
ddev start

Before:

$ ddev start
...
Mutagen sync flush completed in 3s.
For details on sync status 'ddev mutagen st shopware6751 -l'
...

$ ddev mutagen st shopware6751 -l
Failed to get active project: could not find a project named 'shopware6751'. Run 'ddev list' to see currently active projects

After:

$ ddev start
...
Mutagen sync flush completed in 3s.
For details on sync status 'ddev mutagen st shopware6.7.5.1 -l'
...

$ ddev mutagen st shopware6.7.5.1 -l
Mutagen: ok: 
--------------------------------------------------------------------------------
Name: shopware6751
Identifier: sync_166OQQH4yHCVHfSWCt3NWIbW8AqkAJTQrtsbRwN6T6X
Labels:
        com.ddev.config-hash: 2673a03c18e7e55848cc3533573c79b83db662f3
        com.ddev.volume-signature: f91014c1ada1b43a2f08de5ceeb67f07a3568157-1766403112
Configuration:
        Synchronization mode: Two Way Resolved
...

Automated Testing Overview

Release/Deployment Notes

@stasadev stasadev requested a review from a team as a code owner December 22, 2025 11:42
@github-actions
Copy link
Copy Markdown

github-actions bot commented Dec 22, 2025

@rfay rfay force-pushed the 20251222_stasadev_mutagen_st_name branch from 3cbb736 to 7eba177 Compare December 22, 2025 14:46
@rfay
Copy link
Copy Markdown
Member

rfay commented Dec 22, 2025

Rebased

@stasadev
Copy link
Copy Markdown
Member Author

It didn't need a rebase, the change is in text output, not in the code.

Somehow, the rebase removed this commit credit 3cbb736

@rfay
Copy link
Copy Markdown
Member

rfay commented Dec 22, 2025

Sorry, I knew it wouldn't pass all the tests without rebase. Don't know why it would lose credit, but know you'll fix!

Copy link
Copy Markdown
Member

@rfay rfay left a comment

Choose a reason for hiding this comment

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

This is great, thanks.

I tested with touch web/junk.php && chmod 000 web/junk.php

I tested with a project named d.11

Without this, the incorrect info was given:

Starting Mutagen sync process....
Mutagen sync completed with problems in 3s.
For details on sync status 'ddev mutagen st d11 -l'

With this, the correct info was given.

I note that there are other minor problems with the dot removal to pacify docker-compose. (Not saying it's worth looking at.)

ddev stop --unlist d11 stops the project d.11 but does not unlist it :) It successfully unlists d11

@rfay rfay merged commit a0d7d00 into ddev:main Dec 23, 2025
23 of 25 checks passed
@rfay rfay deleted the 20251222_stasadev_mutagen_st_name branch December 23, 2025 02:54
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants