I wanted to get your input on the changes made on src/phorkie/SetupCheck.php:
776fadd - I filed a issue with the author of PHP-Markdown (https://github.com/michelf/php-markdown/pull/46#issuecomment-8869844) because it does not confirm to PEAR naming conventions for the class and filename. It does not autoload. Therefore, I'm removing it temporarily from the index check.
494f8ff - Added (cursory) checking for a valid elastic search url. You MUST use a namespace (e.g /phorkie/) on the url, you cannot use the root directory (i.e. /). This isn't obvious in the README, and the cause of my elasticsearch woes.
12f8fc8 - Updated README.
Once PHP-Markdown fixes its issue, I propose making scripts/index.php a more generic "run this script after setup" rather than a "elasticsearch index script", which should be run inbetween step 5 and 6 during installation.
I wanted to get your input on the changes made on
src/phorkie/SetupCheck.php
:/phorkie/
) on the url, you cannot use the root directory (i.e./
). This isn't obvious in the README, and the cause of my elasticsearch woes.Once PHP-Markdown fixes its issue, I propose making
scripts/index.php
a more generic "run this script after setup" rather than a "elasticsearch index script", which should be run inbetween step 5 and 6 during installation.