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鈥檒l occasionally send you account related emails.

Already on GitHub? Sign in to your account

Add 'webrick' warning note to "Quickstart" Docs #8727

Merged
merged 1 commit into from Jul 22, 2021

Conversation

miguelbrandao
Copy link
Contributor

This is a 馃敠 documentation change.

Summary

For Ruby 3.0.0 and higher, bundle exec jekyll serve will fail because of a missing dependency: webrick.

The documentation doesn't mention this problem, while it states that all Ruby versions above 2.4.0 are supported. This is confusing to new users who will probably install the latest Ruby version and thus will be unable to move past step 5 of the Quickstart.

This PR aims to put a band-aid in this situation by adding a warning at the end of the instructions, allowing affected users to quickly identify and solve the problem.

image

Context

I came across this problem when starting a new site and getting a bunch of errors on step 5 of the Quickstart.
A quick search lead me to this discussion which explained what was happening and provided a working solution.

@hendramanu
Copy link

hendramanu commented Jul 6, 2021

Agree. It should included in docs. Almost all Jekyll template I found were getting webrick error.

@DirtyF
Copy link
Member

DirtyF commented Jul 22, 2021

@jekyll: merge +docs

@jekyllbot jekyllbot merged commit bcaf878 into jekyll:master Jul 22, 2021
jekyllbot added a commit that referenced this pull request Jul 22, 2021
github-actions bot pushed a commit that referenced this pull request Jul 22, 2021
Miguel Brand茫o: Add 'webrick' warning note to "Quickstart" Docs (#8727)

Merge pull request 8727
@jekyll jekyll locked and limited conversation to collaborators Jul 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