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

Step By Step Instructions Review #8399

Merged
merged 6 commits into from
Jan 19, 2021

Conversation

shannonxtreme
Copy link
Contributor

This is a 🔦 documentation change.

Summary

Reworking the step-by-step documentation for clarity, flow, and accuracy.

…n page - Update instruction text - Change content for consistency and clarity - Add note info classes where appropriate - Simplify the content
 - For setup, add info  about livereload option.
 - For liquid, edit content for clarity and better tutorial flow
@shannonxtreme
Copy link
Contributor Author

@DirtyF draft request for changes to tutorials section. I'll undraft when all topics are ready for review.

Copy link
Member

@DirtyF DirtyF left a comment

Choose a reason for hiding this comment

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

Awesome, thanks for the review 🙏

 - Refined content on the front matter page and added a Note
 - Updated flow and content on layouts page
@shannonxtreme
Copy link
Contributor Author

shannonxtreme commented Nov 15, 2020 via email

@DirtyF DirtyF marked this pull request as ready for review January 19, 2021 08:55
@DirtyF
Copy link
Member

DirtyF commented Jan 19, 2021

Let's publish this.

@jekyll: merge +docs

@jekyllbot jekyllbot merged commit d46897f into jekyll:master Jan 19, 2021
jekyllbot added a commit that referenced this pull request Jan 19, 2021
github-actions bot pushed a commit that referenced this pull request Jan 19, 2021
Shannon Kularathna: Step By Step Instructions Review (#8399)

Merge pull request 8399
github-actions bot pushed a commit to chauncey-garrett/jekyll that referenced this pull request Jan 19, 2021
jekyllbot: Update history to reflect merge of jekyll#8399 [ci skip]
github-actions bot pushed a commit to X-EcutiOnner/jekyll that referenced this pull request Jan 19, 2021
jekyllbot: Update history to reflect merge of jekyll#8399 [ci skip]
github-actions bot pushed a commit to patilswapnilv/jekyll that referenced this pull request Jan 19, 2021
jekyllbot: Update history to reflect merge of jekyll#8399 [ci skip]
github-actions bot pushed a commit to tigefa4u/jekyll that referenced this pull request Jan 19, 2021
jekyllbot: Update history to reflect merge of jekyll#8399 [ci skip]
@jekyll jekyll locked and limited conversation to collaborators Mar 22, 2022
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

4 participants