Skip to content

Conversation

@myang1010
Copy link
Owner

@myang1010 myang1010 commented Apr 21, 2021

Student Ming King
Submit week5 assignment.
Thank you!

@myang1010 myang1010 changed the title submit week4 assignment submit week5 assignment Apr 21, 2021
Copy link

@franchizi franchizi left a comment

Choose a reason for hiding this comment

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

You're doing a great job!


def greet(name)
"Hello, #{name}!"
end

Choose a reason for hiding this comment

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

This is correct.


def ctof(temperature)
return temperature * 9.0 / 5 + 32
end

Choose a reason for hiding this comment

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

This is good also.

result*= (number+1)
end
return result
end

Choose a reason for hiding this comment

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

Well done!

it "computes the factorial of 10" do
expect(factorial(10)).to eq(3628800)
end
end

Choose a reason for hiding this comment

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

This is properly done!

expect(s).to eq("Ahay! Iay otgay ouyay!")
end

end

Choose a reason for hiding this comment

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

Good job!

# end
# end
# return c
# end

Choose a reason for hiding this comment

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

The code in the title method looks like it would work, but how are you getting input (as in a string) for the method to operate on?

it "doesn't pad a two-digit number" do
expect(@timer.padded(12)).to eq('12')
end
end

Choose a reason for hiding this comment

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

This is good.

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.

3 participants