GHC 2016-12-14

9 comments.

, https://git.io/v1Dj5 in zmwangx/gharchive
Also, we want to record the ETag in a new column to reduce transfer time and impact on the rate limit.

, https://git.io/v1Dj7 in zmwangx/gharchive
Might want to explore providing an experimental fetching interface with the GraphQL API.

, https://git.io/v1DMU in mpv-player/mpv
Done (and tested): https://github.com/mpv-player/mpv.io/pull/45.

, https://git.io/v1DMT in mpv-player/mpv.io
Gemfile.lock: update eventmachine and json
==========================================

The previous versions of eventmachine and json could not be installed on macOS 10.12.2 due to failures during native extension compilation.

Updating them to the latest patch version of the respective major.minor version.

, https://git.io/v1DMk in mpv-player/mpv.io
build_mpv_manual: strip toc when calling rst2html
=================================================

A TOC is being introduced to `mpv.rst` in mpv-player/mpv#3906. If merged, we need to strip the TOC from the manuals hosted on mpv.io, because we already have a navigation sidebar that serves as a sticky, interactive TOC.

, https://git.io/v1Dwm in mpv-player/mpv
`--strip-elements-with-class=contents` should be added to the `rst2html` calls in `Rakefile`, too. https://github.com/mpv-player/mpv.io/blob/master/Rakefile#L9-L29. Beyond that it should be no different IFAICT.

, https://git.io/v1DTJ in zmwangx/gharchive
Refresh comments on demand
==========================

I edit my comments a lot, but edits do not show up in my activity feed. I should add a refresh action to `zmwangxsay`, and possibly also expose it in `build`.

Annoying limitations:

1. We need to add an `api_url` column and migrate the schema again;
2. We need to figure out a way to extract `api_url` from existing `url`s;
3. We need to make one request for each comment, subject to rate limiting rules, so we need to limit ourselves to the last N comments.

, https://git.io/v119N in mpv-player/mpv
@wiiaboo Good call, thanks!

, https://git.io/v11io in mpv-player/mpv
manpage: add table of contents to the HTML version
==================================================

The HTML manual is really long, so a TOC really helps:

![mpv html-with-toc](https://cloud.githubusercontent.com/assets/4149852/21165226/1488ba6a-c16c-11e6-9a47-a615516bcb47.png)

I'm not limiting the TOC depth at the moment, but we can easily add `depth` to the `contents` directive.

<del>Unfortunately the `contents` directive is processed by both `rst2man` and `rst2html`, and my current way to keep it out of `mpv.1` is pretty ugly. It would be great if someone could think of a better approach.</del>

---

I agree that my changes can be relicensed to LGPL 2.1 or later.