-
Notifications
You must be signed in to change notification settings - Fork 30
Fixed some typos #1
Conversation
* Username/Id: `csarrazi/7494d27255d0561157b8` | ||
* Gist URI: `https://gist.github.com/csarrazi/7494d27255d0561157b8` | ||
* Gist Id: `565752f13499a3fa17d9` | ||
* Username/Id: `csarrazi/565752f13499a3fa17d9` |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
lyrixx
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
You are fast ;) Fixed. Thanks.
it would be good to fix the runner to support HHVM properly |
and no, allowing failure on HHVM is not the right way |
@stof huhu. I don't have HHVM installed on my laptop :( So it's not going to be easy to fix it. More over, it looks like it's a travis issue. Then, IMHO, HHVM support could be added into another PR. |
👍 |
-1 for the allowed failure. instead, you should fix
|
@@ -8,13 +8,13 @@ You can easily [create a gist](https://gist.github.com) to share a snippet and | |||
execute it using `melody`. Instead of downloading the file to your computer, | |||
simply pass the URL to `melody`: | |||
|
|||
$ melody run https://gist.github.com/csarrazi/7494d27255d0561157b8 | |||
$ melody run https://gist.github.com/lyrixx/565752f13499a3fa17d9 |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
\o/
@stof I'm going to merge this one because the HHVM support could and will be added by another PR. Then this PR really have to be merged to avoid angry user and to avoid duplicate effort |
This PR was merged into the master branch. Discussion ---------- Fixed some typos Commits ------- de2a4ba Fixed some code issue (thanks SensioLabsInsight) e20c1e6 Fixed composer vendor dir detection when the composer binary is outdated 3346502 [Travis] Allow failure with hhvm e49d261 [Travis] Fixed configuration 156d484 Updated vendor 4e284ce Fixed tests dcc2958 [Doc] Fixed wrong doc (old format) c2af15c Fixed bad copy/paste
No description provided.