8000 Blog: Manually managing tool versions and adding custom utilities by bserem · Pull Request #382 · ddev/ddev.com · GitHub
[go: up one dir, main page]
More Web Proxy on the site http://driver.im/
Skip to content

Blog: Manually managing tool versions and adding custom utilities #382

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Merged
merged 22 commits into from
Jun 30, 2025

Conversation

bserem
Copy link
Contributor
@bserem bserem commented Jun 12, 2025

The Issue

Blog about manually managing installed software in ddev

How This PR Solves The Issue

Provides the article

Manual Testing Instructions

Automated Testing Overview

Related Issue Link(s)

ddev/ddev#7352

Release/Deployment Notes

@rfay
Copy link
Member
rfay commented Jun 12, 2025

Just a note: Please use a meaningful branch name (not main) when creating a PR anywhere. If you use main it means you can only ever have one PR, and anything you do on main breaks the existing PR.

Copy link
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.

Definitely needs to lead with the problem at hand (terminus downgrade) and that needs to be added to the title as well.

@bserem
Copy link
Contributor Author
bserem commented Jun 12, 2025

Changes applied!

I'll remember the tip for branching.

@bserem bserem requested a review from rfay June 12, 2025 14:56
@rfay rfay added this to Next TODOs Jun 13, 2025
Copy link
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.

Coming along great, thanks for doing this!

@bserem
Copy link
Contributor Author
bserem commented Jun 19, 2025

Pushed some changes, please review.

Thanks for holding my hand in this one

@rfay
Copy link
Member
rfay commented Jun 23, 2025

On my list, thanks! If you get a chance also try to fix the test failure (should be easy, hints on lint fixes in the README.md)

Copy link
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.

Thanks for the work on this! I'll see if I can clear up the linting failures.

Copy link
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.

Looking good, thanks! I fixed the prettier linting complaint so do a git pull if editing locally.

@rfay rfay requested review from tyler36 and stasadev June 25, 2025 14:08
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.

Copy link
Contributor
@tyler36 tyler36 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.

Co-authored-by: tyler36 <tyler36@users.noreply.github.com>
bserem and others added 5 commits June 30, 2025 13:58
Co-authored-by: Randy Fay <randy@randyfay.com>
Co-authored-by: Randy Fay <randy@randyfay.com>
Co-authored-by: Randy Fay <randy@randyfay.com>
Co-authored-by: Randy Fay <randy@randyfay.com>
Co-authored-by: Randy Fay <randy@randyfay.com>
@bserem
Copy link
Contributor Author
bserem commented Jun 30, 2025

Thanks all for the review, and the help on English language!

Copy link
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.

Yay, thanks!

@rfay rfay merged commit 984f9a2 into ddev:main Jun 30, 2025
1 check passed
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
0