Skip to content

Conversation

@preeti-t
Copy link

Self checklist

  • I have titled my PR with Region | Cohort | FirstName Last Name | Sprint | Assignment Title
  • My changes meet the requirements of the task
  • I have tested my changes
  • My changes follow the style guide

Change list

Briefly explain your PR.

Questions

Ask any questions you have for your reviewer.

completed the code using for loop
made changes as required
completed the code
completed the code for different cases
made correction
completed the cases
Copy link

@MustafaBora MustafaBora left a comment

Choose a reason for hiding this comment

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

Check my comment

@@ -1,5 +1,11 @@
function getOrdinalNumber(num) {
if (num > 1 || num < 1) {

Choose a reason for hiding this comment

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

Can you implement this please?

Copy link
Author

Choose a reason for hiding this comment

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

I have added implementation in the code. The code is now updated @MustafaBora

Choose a reason for hiding this comment

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

I see you have solved the question on get-ordinal-number.test.js.
Test file should contain only the tests, and this file should contain the solution

Choose a reason for hiding this comment

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

tests like:
test("should return '3rd' for 3", () => {
expect(getOrdinalNumber(3)).toEqual("3rd");
});

test("should return '11th' for 11", () => {
expect(getOrdinalNumber(11)).toEqual("11th");
});

test("should return '13th' for 13", () => {
expect(getOrdinalNumber(13)).toEqual("13th");
});

implementation completed.
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