Confusing explanation in Part 2 - The shortest person in the room

Open
#15 0 comments 0 reactions 0 assignees View on GitHub

Nobody has claimed this yet.

Assessment

Difficulty
1/5
Estimated time
Under an hour
Newbie friendliness
55/100
Issue type
Documentation
Clarity
Mostly clear
Activity status
Stale
Tech stack
javascript
Domain
documentation

Research direction

Start with the Part 2 exercise text for “The shortest person in the room” and its testing-code example. Update the documented output to show the shortest person's name and height, matching the expected Person return type; done when the example no longer suggests returning only person.name.

Written by the indexing model from the issue text.

Description

There is a confusing information in the part 2 of the exercise called "The shortest person in the room".

The issue is regarding the output we should get when introducing the testing code in the exercise. The example output says it should print a string with the name of the shortest person (In this example Nina) which leads to conclusion that the method should return person.name attribute, but when doing so TMC shows an error and says the shortest() method should return a Person type data which it can never print just the name attribute unless specified as print(person.name)

The output of testing the part 2 of the exercise shouldn't be "Shortest: Nina", it should actually be "Shortest: Nina (162 cm)"

Dominant language
JavaScript
Stars
77
Forks
30
PR merge metrics
No merged PRs in 30d

Contributor guide

No contributing guide indexed for this repository

First steps

  1. Read the whole issue, then the project's contributing guide.
  2. Comment on the issue to say you are picking it up — it saves two people doing the same work.
  3. Fork the repository and make your change on a branch.
  4. Open a pull request that references the issue number.

More from rage/programming-26

All issues in rage/programming-26

Similar issues

More JavaScript issues

Get new issues in your inbox

A short digest of beginner-friendly GitHub issues.