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.

Sign up for GitHub

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

Jump to bottom

Prevent VM_DIR and VVV_PATH_TO_SITE having trailing slashes #2731

Merged
merged 2 commits into from
Sep 18, 2024
Merged

Prevent VM_DIR and VVV_PATH_TO_SITE having trailing slashes #2731

merged 2 commits into from
Sep 18, 2024

Conversation

tomjn
Copy link
Member

@tomjn tomjn commented Sep 18, 2024 •
edited
Loading

Fixes a bug introduced in abea1f5

Related to #2729

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.

@update-docs Update Docs
Copy link

update-docs bot commented Sep 18, 2024 •
edited by unfurl-links bot
Loading

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 mentioned this pull request Sep 18, 2024
Closed
@tomjn tomjn marked this pull request as ready for review September 18, 2024 14:36
@tomjn tomjn requested a review from a team September 18, 2024 14:36
@tomjn tomjn added this to the 3.14 milestone Sep 18, 2024
@tomjn tomjn merged commit 0ce4b25 into develop Sep 18, 2024
3 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@Mte90 Mte90 Mte90 approved these changes

Assignees
No one assigned
Labels
needs:testing priority:high provisioner:core
Projects
None yet
Milestone
3.14
Development

Successfully merging this pull request may close these issues.
2 participants
@tomjn @Mte90