Skip to content

[WIP] mago#283

Draft
tangrufus wants to merge 4 commits into
mainfrom
mago
Draft

[WIP] mago#283
tangrufus wants to merge 4 commits into
mainfrom
mago

Conversation

@tangrufus

Copy link
Copy Markdown
Member

No description provided.

Copilot AI review requested due to automatic review settings July 24, 2026 19:49

Copilot AI left a comment

Copy link
Copy Markdown

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 updates the mago tool version used by local tooling and CI, and refreshes the Composer dependency lockfile accordingly.

Changes:

  • Bump mago from 1.30 → 1.42 in mise.toml.
  • Pin mago version to 1.42 in the PHP lint GitHub Actions workflow.
  • Update composer.lock (notably including a Guzzle major-version change).

Reviewed changes

Copilot reviewed 2 out of 2 changed files in this pull request and generated no comments.

File Description
mise.toml Updates the mago tool version used by mise tasks.
composer.lock Updates locked PHP dependencies (includes a major Guzzle upgrade).
.github/workflows/lint.yml Aligns CI lint workflow inputs with the updated mago version.

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

Copilot AI review requested due to automatic review settings July 24, 2026 19:51

Copilot AI left a comment

Copy link
Copy Markdown

Choose a reason for hiding this comment

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

Pull request overview

Copilot reviewed 2 out of 2 changed files in this pull request and generated 2 comments.

Comment on lines 25 to 31
with:
phpstan: true
mago: true
version: '1.42'
box: true
box-php-version: '8.5'
box-version: '4'
Comment thread mise.toml

[tools]
mago = "1.30" # Must be kept in sync with @.github/workflows/lint.yml
mago = "1.42" # Must be kept in sync with @.github/workflows/lint.yml
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Development

Successfully merging this pull request may close these issues.

2 participants