Skip to content

[self-update] Restore global new-version notifications (#300)#301

Merged
coisa merged 2 commits intomainfrom
bugfix/300-version-check-notification
Apr 30, 2026
Merged

[self-update] Restore global new-version notifications (#300)#301
coisa merged 2 commits intomainfrom
bugfix/300-version-check-notification

Conversation

@coisa
Copy link
Copy Markdown
Contributor

@coisa coisa commented Apr 30, 2026

Related Issue

Closes #300

Motivation / Context

  • Global DevTools installs could stay silent even when a newer release was available because the version checker compared the Composer lookup result against a Symfony Process success constant that is not available in the supported runtime range.

Changes

  • Replaced the version-check success comparison with Process::isSuccessful() after the Composer metadata lookup runs.
  • Added ComposerVersionCheckerTest coverage for successful and failing Composer release lookups.
  • Recorded the fix in CHANGELOG.md.

Verification

  • composer dev-tools
  • Focused command(s): composer dev-tools tests -- --filter='ComposerVersionCheckerTest'
  • Manual verification: not rerun after the patch; the regression is covered by the new checker test and the full DevTools gate.

Documentation / Generated Output

  • README updated
  • docs/ updated
  • Generated or synchronized output reviewed

Changelog

  • Added a notable CHANGELOG.md entry

Reviewer Notes

  • The local .github/wiki modification was preserved outside this branch and is not part of this PR.

@github-actions
Copy link
Copy Markdown
Contributor

@coisa coisa merged commit dae7a7d into main Apr 30, 2026
12 checks passed
@coisa coisa deleted the bugfix/300-version-check-notification branch April 30, 2026 03:35
@github-project-automation github-project-automation Bot moved this from Backlog to Released in PHP Fast Forward Project Apr 30, 2026
github-actions Bot added a commit that referenced this pull request Apr 30, 2026
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

bug Something isn't working

Projects

Status: Released

Development

Successfully merging this pull request may close these issues.

Restore new-version notifications for global installs

1 participant