Conversation
@kevinsawicki can you peep this? |
bump @kevinsawicki |
@@ -46,6 +46,8 @@ var fetchDocs = module.exports = function fetchDocs (_settings, callback) { | |||
|
|||
// override tarball for testing out branches: | |||
// tarballUrl = 'https://api.github.com/repos/electron/electron/tarball/master' | |||
// tarballUrl = 'https://github.com/electron/electron/archive/ba49fbc8ba2fac0b72b1530f2ce46c1cd3110620.tar.gz' | |||
// version = 'v1.4.1' |
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 was a little confused by this line, is it meant to be uncommented when you want to override it? It seems to get reassigned two lines below.
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 point. Cleaned that up a bit. I just left it there because I always have a hard time finding/remembering the URL structures for fetching repo tarballs.
<td>{{ site.data.processes[doc.title] | replace: 'Process', '' | replace: 'Processes', '' }}</td> | ||
<td>{{ doc.excerpt }}</td> | ||
</tr> | ||
{% unless doc.source_url contains '/structures/' %} |
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.
Instead of indenting here, we could switch to using continue
statements so the <tr>
is more aligned with the outer <table>
element:
{% assign docs = site.docs | sort: 'sort_title' %}
{% for doc in docs %}
{% if doc.category != 'API' %}{% continue %}{% endif %}
{% if doc.source_url contains '/structures/' %}{% continue %}{% endif %}
<tr>
<td><a href="{{ site.baseurl }}{{ doc.url }}">{{ doc.title }}</a></td>
<td>{{ site.data.processes[doc.title] | replace: 'Process', '' | replace: 'Processes', '' }}</td>
<td>{{ doc.excerpt }}</td>
</tr>
{% endfor %}
Curious what you think.
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.
Like early returns. I dig it.
Looks good to me 👍 |
Fixes #509
This updates the markdown parser to rewrite links to the new files in
/docs/api/structures
.cc @MarshallOfSound