Skip to content

New bugfix and additional necessities for Exercise: 01-greet-people#88

Open
BarzinDevelops wants to merge 3 commits intoCodeYourFuture:masterfrom
BarzinDevelops:new_bugfix_greetPeople
Open

New bugfix and additional necessities for Exercise: 01-greet-people#88
BarzinDevelops wants to merge 3 commits intoCodeYourFuture:masterfrom
BarzinDevelops:new_bugfix_greetPeople

Conversation

@BarzinDevelops
Copy link

@BarzinDevelops BarzinDevelops commented Oct 10, 2022

Changed the function so it does return an array of each name prefixed with 'Hello' ->
like this: ['Hallo name1', 'Hello name2', etc..]

Before this change the function returned: ['hello name1name2name3] which wasn't the desired result to expect.

The MISSING necessities of each Assignment/Exercise:

  • 1. Each assignment/exercise should have a description file so developers know what is expected.
  • 2. Also alway add a solution so codes can be compared.

For this exercise (greet-people) i created those files as example.

  • Added description for greet-people.
  • Added solution file for greet-people.

View rendered II.write-tests/01-greet-people/Exercise_Description.md

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.

1 participant