Skip to content

Conversation

@rfay
Copy link
Member

@rfay rfay commented Sep 22, 2025

The Issue

How This PR Solves The Issue

  • Update Node.js from 18 to 24 in dependabot workflow
  • Update matrix OS from ubuntu-22.04 to ubuntu-24.04-arm
  • Update DDEV test version from v1.24.0 to v1.24.7
  • Replace ubuntu-latest with ubuntu-24.04 for consistency

Manual Testing Instructions

Review results

Automated Testing Overview

Related Issue Link(s)

🤖 Generated with Claude Code

- Update Node.js from 18 to 24 in dependabot workflow
- Update matrix OS from ubuntu-22.04 to ubuntu-24.04-arm
- Update DDEV test version from v1.24.0 to v1.24.7
- Replace ubuntu-latest with ubuntu-24.04 for consistency
- Extract DDEV version to OLD_DDEV_VERSION environment variable

🤖 Generated with [Claude Code](https://claude.ai/code)

Co-Authored-By: Claude <[email protected]>
Remove environment variable approach and use v1.24.7 directly throughout
the workflow for clarity and simplicity.

🤖 Generated with [Claude Code](https://claude.ai/code)

Co-Authored-By: Claude <[email protected]>
@rfay
Copy link
Member Author

rfay commented Sep 22, 2025

The tests here don't actually check the PR because they use the workflows from HEAD, but I pushed this to ddev-test/github-action-setup-ddev and they seem to work fine:

https://github.com/ddev-test/github-action-setup-ddev/actions/runs/17921085569

@rfay rfay requested a review from stasadev September 22, 2025 16:09
Copy link
Member

@stasadev stasadev left a comment

Choose a reason for hiding this comment

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

Looks good to me.

@rfay rfay added this to Next TODOs Sep 22, 2025
@rfay rfay moved this to In Progress in Next TODOs Sep 22, 2025
@jonaseberle
Copy link
Member

Thanks!

@jonaseberle jonaseberle merged commit f798903 into main Sep 23, 2025
8 checks passed
@jonaseberle jonaseberle deleted the 20250922_rfay_stop_22.04 branch September 23, 2025 06:16
@github-project-automation github-project-automation bot moved this from In Progress to Done in Next TODOs Sep 23, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

Status: Done

Development

Successfully merging this pull request may close these issues.

4 participants