Skip to content
This repository has been archived by the owner on Mar 12, 2019. It is now read-only.

Tutorial fixes #32

Merged
merged 3 commits into from Mar 20, 2016
Merged

Tutorial fixes #32

merged 3 commits into from Mar 20, 2016

Conversation

des4maisons
Copy link
Contributor

After going through the tutorial, here are some improvements I would make to the docs:

  • Use diffs the whole way through to show what code should be updated
  • Clarify that the !command help text is coming from the plugin
  • Change the "hacker news" timer to run every minute instead of 5, to lessen the time needed to see results.

@des4maisons
Change code blocks in tutorial to diffs
accd938 
This makes it clearer what is being added/removed, where.
@des4maisons
Specify which help output comes from plugin, in plugin tutorial
6f09568
@des4maisons
Print HN articles every 1 min instead of 5 in tutorial
6b49b1e 
This makes debugging easier, because the event happens more frequently.
@marvinpinto
Copy link
Owner

Looks great, thank you! 👍

marvinpinto added a commit that referenced this pull request Mar 20, 2016
@marvinpinto
Merge pull request #32 from des4maisons/tutorial-fixes
490149c 
Tutorial fixes
@marvinpinto marvinpinto merged commit 490149c into marvinpinto:master Mar 20, 2016
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Reviewers
No reviews
Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
2 participants
@des4maisons @marvinpinto