Skip to content
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

Better PHP warnings and IP6 VM hostfiles #2689

Merged
merged 4 commits into from
Oct 16, 2023
Merged

Conversation

tomjn
Copy link
Member

@tomjn tomjn commented Oct 15, 2023

When VVV detects the requested PHP version is missing, remind the user they need to install it themselves via config.yml and link out to the PHP changing doc

Also add ::1 too when updating the VMs hosts file

Checks

  • I've updated the changelog.
  • I've tested this PR
  • This PR is for the develop branch not the stable branch.
  • This PR is complete and ready for review.

When VVV detects the requested PHP version is missing, remind the user they need to install it themselves via config.yml and link out to the PHP changing doc

Also add `::1` too when updating the VMs hosts file
@update-docs
Copy link

update-docs bot commented Oct 15, 2023

Thanks for opening this pull request! Make sure CHANGELOG.md gets updated with this change, additionally any docs that need updated can be found at https://github.com/Varying-Vagrant-Vagrants/varyingvagrantvagrants.org

GitHub
The VVV docs and website. Contribute to Varying-Vagrant-Vagrants/varyingvagrantvagrants.org development by creating an account on GitHub.

@tomjn tomjn marked this pull request as ready for review October 15, 2023 10:59
@tomjn tomjn requested a review from a team October 15, 2023 11:00
Mte90
Mte90 previously approved these changes Oct 16, 2023
@tomjn tomjn added this to the 3.13 milestone Oct 16, 2023
@tomjn tomjn merged commit ac78a24 into develop Oct 16, 2023
1 of 5 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants