Run git-diff after dumping the database to make sure that there are no
changes to the csv files.
We often get PRs from people who edit stuff by hand and don't know to do
a load/dump of the database afterwards. Usually this means they were
adding game data by hand, which we want to discourage, but even if it's
done for valid reasons it's good to canonicalize the csv files as soon
as possible to avoid interfering with future commits.
The help text for pokedex dump -l says
-l LANGS, --langs LANGS
comma-separated list of language codes to load,
'none', or 'all' (default: en)
However, the values 'none' and 'all' don't work (they are treated as
language names) and the default is to dump all languages, not just
English.
So `dump -l all` actually dumps no text at all because there is no
language called 'all'.
This is about to cause trouble for Travis CI because the next commit
will start checking that load/dump successfully round-trips... which
definitely isn't the case right now because the dump is missing
all text from translated tables.
The dump command and its documentation should definitely be fixed at
some point, but for now just stop using -l.
`sudo: false` used to be the way to opt in to their container-based
infrastructure. That system was retired in 2018 and so the `sudo` option
no longer does anything. Remove it.
Travi CI supports multiple OSes now. Add `os: linux` to be explicit
about which OS we want to test on.
Markdown keeps breaking the 2.6 build. We could pin an earlier version,
or we can simply drop 2.6 support. The Python world is moving on; time
for us to do the same.
This will apparently allow us to build on Travis's new container-based build
system, which is supposed to lead to faster builds. The catch is that we
don't get root access, but we don't need it anyway.
http://docs.travis-ci.com/user/migrating-from-legacy/
Andrew Ekstedt