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

[Docs] Remove vagrant references #10076

Merged
merged 1 commit into from
Jan 9, 2019
Merged

[Docs] Remove vagrant references #10076

merged 1 commit into from
Jan 9, 2019

Conversation

lchrusciel
Copy link
Member

Q A
Branch? 1.2
Bug fix? no
New feature? no
BC breaks? no
Deprecations? no
Related tickets
License MIT

The vagrant box is outdated and not maintained. I will not find time in the nearest future to pick it up. I want to remove Vagrant references, so it will not be misleading for newcomers. Also, if there is some good spirit, who would like to maintain our Vagrant box, we would be more than happy to transfer the reposiotry itself. Otherwise it will be abandon/read-only.

Also, I owe they a 🍺 ;)

@lchrusciel lchrusciel requested a review from a team as a code owner January 8, 2019 09:38
@Zales0123 Zales0123 added the Documentation Documentation related issues and PRs - requests, fixes, proposals. label Jan 8, 2019
@pamil pamil merged commit 4e7608e into Sylius:1.2 Jan 9, 2019
@pamil
Copy link
Contributor

pamil commented Jan 9, 2019

Thanks, Łukasz! 🎉

@lchrusciel lchrusciel deleted the vagrant branch January 9, 2019 09:35
@prigal
Copy link

prigal commented Jan 22, 2019

Was reviewing 1.3.7 changelog and found this. If needed, you can send users to Homestead like Symfony do in its documentation. I have tested it yesterday for a quick plugin install on a clean sylius standard and it just work.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Documentation Documentation related issues and PRs - requests, fixes, proposals.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

5 participants