Skip to content

Fix typos in Readme #19

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Merged
merged 1 commit into from
May 31, 2019
Merged

Conversation

zeckdude
Copy link
Contributor

@zeckdude zeckdude commented May 31, 2019

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
Member

1. The slug in the `./bin/analyze.sh` example is incorrect

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

Copy link
Member

@tejasbubane tejasbubane left a comment

Choose a reason for hiding this comment

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

Thanks!

@zeckdude
Copy link
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
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