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

Correct PHP example. Add a requirements section. #123

Merged

Conversation

taz77
Copy link
Contributor

@taz77 taz77 commented Jul 18, 2016

Corrected PHP example for missing semi-colons. Added a new requirements section that explains the need for composer and the need of the PHP Intl library to use the spoof validation.

Corrected PHP example for missing semi-colons. Added a new requirements section that explains the need for composer and the need of the PHP Intl library to use the spoof validation.
@@ -4,6 +4,10 @@
With the help of

![Powered by PhpStorm](https://www.jetbrains.com/phpstorm/documentation/docs/logo_phpstorm.png)

##Requirements##
To load this library, you must have [Composer](https://getcomposer.org) installed and operational. The Spoofchecking validation requires that your PHP system have the [PHP Internationalization Libraries](http://php.net/manual/en/book.intl.php) (also known as PHP Intl) insalled.
Copy link
Owner

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Can you please make it a list?
Also add a link to the Spoof validation or to the list of available validations.

Turned requirements into an unordered list. Removed the PHPstorm logo as it doesn't exist and could not find one to replace it.
@taz77
Copy link
Contributor Author

taz77 commented Jul 19, 2016

@egulias Updated. Removed the PHPStorm logo as it was a broken link and I cannot find a replacement. Jetbrains has a link to a zip file of logos on their press page but not the individual image files.

@egulias
Copy link
Owner

egulias commented Jul 20, 2016

Yeah, I see they've unified all the logos.


![Powered by PhpStorm](https://www.jetbrains.com/phpstorm/documentation/docs/logo_phpstorm.png)
Copy link
Owner

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

At least we can leave the phrase and a link to the PHPStrom page.

@egulias egulias merged commit 9160e26 into egulias:master Aug 7, 2016
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants