Skip to content
This repository has been archived by the owner on Mar 5, 2019. It is now read-only.

Feature/vagrant #51

Open
wants to merge 4 commits into
base: master
Choose a base branch
from
Open

Feature/vagrant #51

wants to merge 4 commits into from

Conversation

eddiejaoude
Copy link
Contributor

@eddiejaoude eddiejaoude commented Oct 18, 2016 •
edited by chrisyeoward
Loading

DO NOT MERGE

  • Vagrant setup
  • PHP installed
  • Ruby installed
  • Docker
  • PHP tested with existing code
  • Ruby tested with existing code
  • Python tested with existing code

@chrisyeoward
Copy link
Contributor

The php script works, but the version of it that is in the repo currently needs line 2 editing from <? to <?php in order to work

@eddiejaoude
Copy link
Contributor Author

eddiejaoude commented Oct 19, 2016 •
edited
Loading

Please can someone make that change. Using <? got deprecated a long time ago & is not recommended by the PSR standards for supported versions of PHP, should use full tags of <php more info http://www.php-fig.org/psr/psr-1/

@aspiers aspiers mentioned this pull request Nov 17, 2016
Open
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Reviewers
No reviews
Assignees

@eddiejaoude eddiejaoude

@chrisyeoward chrisyeoward

Labels
DO NOT MERGE
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
2 participants
@eddiejaoude @chrisyeoward