Skip to content

Improve SyncPackagesCommand output, fix navbar dividers, reduce ddev healthcheck interval#155

Merged
LordSimal merged 5 commits into
masterfrom
various-improvements
May 31, 2026
Merged

Improve SyncPackagesCommand output, fix navbar dividers, reduce ddev healthcheck interval#155
LordSimal merged 5 commits into
masterfrom
various-improvements

Conversation

@josbeir
Copy link
Copy Markdown
Contributor

@josbeir josbeir commented Apr 8, 2026

Changes

SyncPackagesCommand — progress & stats output

  • Add a progress bar while processing packages
  • Track and display counters for saved, skipped, and failed packages
  • Add verbose per-package logging ([i/total] package-name)
  • Print a summary after the sync phase and after the stale-package cleanup phase
  • Fix incorrect $package->getName() call in the warning log (was passing the wrong type)

Navbar dropdown dividers

  • Replace the separate <li role="separator"> + <hr> element pattern with a single <li class="border-t border-base-300 pt-1">, using Tailwind/DaisyUI border utilities

ddev phpMyAdmin healthcheck

  • Decrease the healthcheck interval from 120s to 5s so the container is marked healthy much faster on startup

@josbeir josbeir changed the title fix(ui): navbar dropdown dividers Introduces various improvements (wip) Apr 8, 2026
Comment thread .ddev/docker-compose.phpmyadmin.yaml
Comment thread src/Command/SyncPackagesCommand.php
@LordSimal
Copy link
Copy Markdown
Contributor

@josbeir whats the status here?

@josbeir josbeir changed the title Introduces various improvements (wip) Improve SyncPackagesCommand output, fix navbar dividers, reduce ddev healthcheck interval May 30, 2026
@josbeir josbeir marked this pull request as ready for review May 30, 2026 07:41
Copilot AI review requested due to automatic review settings May 30, 2026 07:41
Copy link
Copy Markdown

Copilot AI left a comment

Choose a reason for hiding this comment

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

Pull request overview

This PR improves developer/operator feedback for package syncing, adjusts navbar dropdown divider markup, and speeds up local phpMyAdmin health reporting in DDEV.

Changes:

  • Adds progress, verbose package logging, and sync/cleanup counters to SyncPackagesCommand.
  • Replaces dropdown separator markup with Tailwind/DaisyUI border styling.
  • Reduces the DDEV phpMyAdmin healthcheck interval.

Reviewed changes

Copilot reviewed 3 out of 3 changed files in this pull request and generated 1 comment.

File Description
src/Command/SyncPackagesCommand.php Adds progress output, counters, summaries, and fixes package warning context.
templates/element/navbar.php Updates dropdown divider markup/styling for sign-in/sign-out actions.
.ddev/docker-compose.phpmyadmin.yaml Changes phpMyAdmin healthcheck interval from 120s to 5s.

💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.

healthcheck:
test: ["CMD-SHELL", "true"]
interval: 120s
interval: 5s
@josbeir josbeir force-pushed the various-improvements branch from d0ce2dc to cbee48d Compare May 30, 2026 07:52
@josbeir
Copy link
Copy Markdown
Contributor Author

josbeir commented May 30, 2026

@LordSimal i've updated the pr's title and description, this can be merged now :-)

@LordSimal LordSimal merged commit 13e1e1d into master May 31, 2026
2 checks passed
@LordSimal LordSimal deleted the various-improvements branch May 31, 2026 07:49
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.

4 participants