Skip to content
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

[legacy-framework] docs(tutorial): fix a couple typos and change some copy #344

Merged
merged 3 commits into from
Apr 30, 2020

Conversation

wKovacs64
Copy link
Collaborator

Type: docs

What are the changes and their implications? ⚙️

Fixed a couple typos, formatted a couple things for consistency, generated the directory listing with tree, etc.

Checklist

  • Tests added for changes
  • Any added terminal logging uses packages/server/src/log.ts

Breaking change: no

Other information

@merelinguist Thoughts on these changes? I can remove anything you don't care for, just say the word. Also, I noticed a lot of the punctuation (particularly apostrophes and double quotation marks) are currently represented using codes that are part of the ASCII extended character set rather than the basic set (e.g. <E2><80><99> instead of '). Do we care to change those? It's probably a locale and/or OS thing - not sure how the extended characters look to others.

@wKovacs64 wKovacs64 requested a review from merelinguist April 29, 2020 17:43
This was generated with the following command:

tree mysite --dirsfirst -a -I '.git|.keep'
Copy link
Member

@flybayer flybayer left a comment

Choose a reason for hiding this comment

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

Thank you!

@all-contributors add @wKovacs64 for docs

@flybayer flybayer merged commit 44c4883 into canary Apr 30, 2020
@flybayer flybayer deleted the justin/tutorial branch April 30, 2020 03:45
@wKovacs64
Copy link
Collaborator Author

wKovacs64 commented Apr 30, 2020

Thank you!

@all-contributors add @wKovacs64 for docs

Does this bot fail sometimes? I certainty don't mind, just curious.

Edit: Hah, quoting you triggered it. Maybe it was just distracted earlier.

@allcontributors
Copy link

@wKovacs64

I've put up a pull request to add @wKovacs64! 🎉

@itsdillon itsdillon changed the title docs(tutorial): fix a couple typos and change some copy [legacy-framework] docs(tutorial): fix a couple typos and change some copy Jul 7, 2022
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