Skip to content

Commit ae00374

Browse files
authored
Clarify URL format in instructions.md
Currently, the instructions say to get the URLs, but it doesn't specify what "URLs" are. (https://exercism.org/tracks/javascript/exercises/regular-chatbot is a valid URL, but this exercise does not test for such complicated URLs, and the reference solution in exemplar.js would not reject it.) It would be frustrating for a user to exert their effort trying to write a regex for URLs the exercise isn't testing for (especially if the tests were to reject a valid URL, though I haven't checked if they currently could).
1 parent ca9adcb commit ae00374

File tree

1 file changed

+3
-3
lines changed

1 file changed

+3
-3
lines changed

exercises/concept/regular-chatbot/.docs/instructions.md

Lines changed: 3 additions & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -68,11 +68,11 @@ Example of Conversation:
6868
- **Chatbot**: Hey username, I would like to learn how to code in JavaScript, do you know any cool website where I could learn?
6969
- **User**: I learned a lot from [exercism.org](http://exercism.org)
7070
71-
Implement the function `getURL()` which is able to return an array with just the link of each website.
71+
Implement the function `getURL()` which is able to return an array with just the link of each website. Writing a regex to recognize any URL would be very complicated, but the Chatbot only needs to read simple URLs. For this exercise, a URL is any `"."`-delimited sequence of words.
7272
7373
```javascript
74-
getURL('I learned a lot from exercism.org');
75-
// => ["exercism.org"];
74+
getURL('I learned a lot from exercism.org and developer.mozilla.org, but I found ts39.es very boring!');
75+
// => ["exercism.org", "developer.mozilla.org", "ts39.es"];
7676
```
7777
7878
## Greet the user

0 commit comments

Comments
 (0)