Skip to content

Solving the second task#1

Open
MaximeBaudette wants to merge 18 commits intomasterfrom
example
Open

Solving the second task#1
MaximeBaudette wants to merge 18 commits intomasterfrom
example

Conversation

@MaximeBaudette
Copy link
Owner

This is an example pull request, DO NOT MERGE!

@MaximeBaudette MaximeBaudette self-assigned this Feb 17, 2017
Test 1.txt Outdated
@@ -1 +1,2 @@
1+2= No newline at end of file
1+2=5
Copy link
Contributor

Choose a reason for hiding this comment

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

This is wrong

Copy link
Contributor

@maguilerac maguilerac left a comment

Choose a reason for hiding this comment

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

There is an error on second equations

MaximeBaudette pushed a commit that referenced this pull request Feb 17, 2017
MaximeBaudette pushed a commit that referenced this pull request Feb 17, 2017
Test 1.txt Outdated
1+2= No newline at end of file
1+2=3
2+7=9
10+5=12
Copy link
Contributor

Choose a reason for hiding this comment

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

its wrong

@MaximeBaudette MaximeBaudette changed the title Solving the first task Solving the second task Feb 23, 2018
MaximeBaudette and others added 3 commits February 23, 2018 15:03
Also I added a new equation, let me know if it is correct
Update File 2.txt
@MaximeBaudette
Copy link
Owner Author

See @prottay
Because you submitted your changes to a branch involved in a pending PR, your changes are then integrated in the PR. That's how you can contribute to someone else's PR.

@MaximeBaudette
Copy link
Owner Author

Now I see @prottay that you didn't provide a wrong answer to your simple equation

@MaximeBaudette
Copy link
Owner Author

Now, @mpodlaski see how your contribution is included in the original PR.

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.

6 participants