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

document changes from PR #206 (as per issue #212) #380

Merged
merged 2 commits into from
Jan 8, 2018

Conversation

filipdanic
Copy link
Contributor

Motivation

To get started contributing to Docusaurus, I guess? :)

Test Plan

Build the website to see the changes. (Pretty self-evident what they are.)

Related PRs

This documents the new config field that introduced by #206 and closes the issue #212.

Side note: also added the links that help people find all the themes and languages that are supported by Highlight.js

Copy link
Contributor

@JoelMarcey JoelMarcey left a comment

Choose a reason for hiding this comment

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

@filipdanic Thank you for this pull request. A nice one to get started with. :)

My only nit is that I usually do not link the periods . in a sentence. Also you probably could have just linked the "supported languages here" part. But that is a minor thing and easily fixed.

@JoelMarcey
Copy link
Contributor

Closes #212

@JoelMarcey JoelMarcey merged commit 65085b1 into facebook:master Jan 8, 2018
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
CLA Signed Signed Facebook CLA
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants