-
Notifications
You must be signed in to change notification settings - Fork 4.3k
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
Move the docs to WordPress.org #3381
Conversation
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I fixed a couple typos, other than that this is looking good. Didn't see any missed references to docutron and everything still builds fine. 👍
parents: [ 'outreach' ], | ||
name: 'meetups', | ||
title: 'Meetups', | ||
markdown: require( './talks.md' ), |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Should be meetups.md
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Good catch, thanks @Soean!
Codecov Report
@@ Coverage Diff @@
## master #3381 +/- ##
==========================================
+ Coverage 31.35% 31.44% +0.09%
==========================================
Files 246 248 +2
Lines 6778 7073 +295
Branches 1218 1320 +102
==========================================
+ Hits 2125 2224 +99
- Misses 3915 4042 +127
- Partials 738 807 +69
Continue to review full report at Codecov.
|
Sad to see I think the menu order is not correct. The previous/next links are broken too. |
Might make more sense for |
Good idea, I've update the theme and this PR to do exactly that, it gets the menu order from the |
Docutron was lovely, but it's time to move the docs across to WordPress.org.
The docs will live here: https://wordpress.org/gutenberg/handbook/
WordPress.org will sync the docs from master every 15 minutes (though syncing is currently disabled, until this PR is merged).