exercism / exercism/elixir

log-parser improvements

Open
#1,162 1 comment 0 reactions 0 assignees View on GitHub
Dominant language
Elixir
Stars
664
Forks
416
PR merge metrics
No merged PRs in 30d

Description

There are some improvements to this exercise proposed in the initial PR that got postponed so we can see how the exercise is doing first and maybe collect more improvement ideas first.

## Exercise doesn't teach that different delimiters can be used for regular expressions

Maybe add a step that involves parsing several `/` and ask them to use another delimiter for readability. Can be checked via the analyzer because the delimiter is kept in the AST.

## Exercise doesn't teach about compiling regex with variable content

I wanted to add some step that would require an interpolated regex with escaping a variable, but I couldn't come up with anything thematic that wouldn't be better done with `String.starts_with?`.

How about adding a step to `tag_with_user_name/1?`
Maybe something like bolding the user names whenever they are mentioned without "User"?

```elixir
LogParser.tag_with_user_name("[INFO] User Alice created a new project.")
# => "[USER] Alice [INFO] User Alice created a new project"
LogParser.tag_with_user_name("[INFO] User Alice created a new project. Alice has a reputation of 643.")
# => "[USER] Alice [INFO] User Alice created a new project. **Alice** has a reputation of 643."
```

Contributor guide

Open the contributing guide

Assessment

This issue has not been assessed yet.

Get new issues in your inbox

A short digest of beginner-friendly GitHub issues.