Skip to content

amare16 #1

Open
amare16 wants to merge 13 commits intosimplonco:masterfrom
amare16:master
Open

amare16 #1
amare16 wants to merge 13 commits intosimplonco:masterfrom
amare16:master

Conversation

@amare16
Copy link
Copy Markdown

@amare16 amare16 commented Jul 8, 2016

No description provided.

@yvan-sraka yvan-sraka changed the title amare's first commit on php hello world amare16 Jul 8, 2016
@yvan-sraka
Copy link
Copy Markdown
Contributor

Oups.. I forget to enable Travis CI on this repository, now if you push a commit (like just adding a comment), it will work 😉 !

@amare16
Copy link
Copy Markdown
Author

amare16 commented Jul 9, 2016

Here in github comment place or on terminal?

@yvan-sraka
Copy link
Copy Markdown
Contributor

No, in your files, to do a new "fake" commit 😄 !

@yvan-sraka
Copy link
Copy Markdown
Contributor

Check Travis CI logs: https://travis-ci.org/simplonco/php-hello-world/builds/143839706 😉!

@yvan-sraka
Copy link
Copy Markdown
Contributor

Hint: if you want automated tests working, you have to send only a PHP function without all your HTML test code 😄!

Comment thread style.css
@@ -0,0 +1,8 @@
body {
Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

You can remove this file and the cute image, which are now useless 😉!

@yvan-sraka
Copy link
Copy Markdown
Contributor

Read comments I left for you ❤️!

@amare16
Copy link
Copy Markdown
Author

amare16 commented Jul 12, 2016

What is the problem? it is working on online checker.

Comment thread hello-world.php Outdated
// This is only a SKELETON file for the "Hello World" exercise.
// It's been provided as a convenience to get you started writing code faster.
//
function helloWorld($name) {
Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

Try function helloWorld($name = "") 😉!

@yvan-sraka
Copy link
Copy Markdown
Contributor

CONGRATULATIONS ☀️!

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