Skip to content

Fix typos in Readme#19

Merged
SleeplessByte merged 1 commit into
exercism:masterfrom
zeckdude:feature/fix-typos
May 31, 2019
Merged

Fix typos in Readme#19
SleeplessByte merged 1 commit into
exercism:masterfrom
zeckdude:feature/fix-typos

Conversation

@zeckdude

@zeckdude zeckdude commented May 31, 2019

Copy link
Copy Markdown
Contributor

Fixing a few issues in the README:

  1. The slug in the ./bin/analyze.sh example is incorrect
  2. The path in the ./bin/analyze.sh example should be pointing to the folder instead of the actual solution
  3. The --debug and --console flags need to be in the command if people want to see the output in the terminal, so I think it's helpful to have them in the example

@tejasbubane

Copy link
Copy Markdown
Member
1. The slug in the `./bin/analyze.sh` example is incorrect

Ahh. That's why I faced this - #15 😄

@tejasbubane tejasbubane left a comment

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

Thanks!

@zeckdude

Copy link
Copy Markdown
Contributor Author

You’re welcome. @SleeplessByte added a user friendly message that logged to the console notifying me that the module wasn’t found so it made it easy for me to see the issue. I’m pretty sure he added that in response to your issue.

@SleeplessByte

Copy link
Copy Markdown
Member

Thank you! This is perfect ❤️

@SleeplessByte
SleeplessByte merged commit 8c869f6 into exercism:master May 31, 2019
SleeplessByte added a commit to exercism/typescript-analyzer that referenced this pull request Jun 4, 2019
* Create CODE_OF_CONDUCT (originally @SleeplessByte exercism/javascript-analyzer#5)

* Add dockerfile for automated mentoring support (originally @depsir exercism/javascript-analyzer#12)

* Initial smoke for usage of jest (originally: @SleeplessByte exercism/javascript-analyzer#17)

* Apply design and proper interface and structure (originally: @SleeplessByte exercism/javascript-analyzer#18)

* Fixes issues in the README.md (originally: @zeckdude exercism/javascript-analyzer#19)
SleeplessByte added a commit to exercism/typescript-analyzer that referenced this pull request Jun 4, 2019
* Create CODE_OF_CONDUCT (originally @SleeplessByte exercism/javascript-analyzer#5)

* Add dockerfile for automated mentoring support (originally @depsir exercism/javascript-analyzer#12)

* Initial smoke for usage of jest (originally: @SleeplessByte exercism/javascript-analyzer#17)

* Apply design and proper interface and structure (originally: @SleeplessByte exercism/javascript-analyzer#18)

* Fixes issues in the README.md (originally: @zeckdude exercism/javascript-analyzer#19)
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.

3 participants