Logs for #nikola for 2016-12-07

03:31:24 <verbalshadow> anyone have any idea why the top ul element in not displaying a breadcrumbs like the second one https://transfer.sh/AJZcG/index.html
05:55:50 <ChrisWarrick> verbalshadow: (on mobile, can't see link) do they have the same classes?
17:04:23 -GitHub[nikola]:#nikola- [nikola] Kwpolska closed pull request #2585: Keyword arguments in path handlers (master...kwargs-in-path-handlers) https://git.io/v10tp
17:04:23 -7GHAAWO3W:#nikola- [nikola] Kwpolska pushed 1 new commit to master: https://git.io/v1uPJ
17:04:23 -7GHAAWO3W:#nikola- nikola/master d743c67 Chris Warrick: Merge pull request #2585 from getnikola/kwargs-in-path-handlers...
17:08:55 -travis-ci:#nikola- getnikola/nikola#8032 (master - d743c67 : Chris Warrick): The build passed.
17:08:56 -travis-ci:#nikola- Change view: https://github.com/getnikola/nikola/compare/122380ae737f...d743c673344f
17:08:56 -travis-ci:#nikola- Build details: https://travis-ci.org/getnikola/nikola/builds/182021507
18:23:42 <KwBot> [nikola] felixfontein closed issue #1914: Translatable URL pathnames https://github.com/getnikola/nikola/issues/1914
18:23:57 -GitHub[nikola]:#nikola- [nikola] felixfontein closed pull request #2584: Making INDEX_PATH, RSS_PATH and AUTHOR_PATH translatable (master...translatable-url-pathnames) https://git.io/v14D4
18:24:16 -GitHub[nikola]:#nikola- [nikola] felixfontein pushed 1 new commit to master: https://git.io/v1uhq
18:24:16 -GitHub[nikola]:#nikola- nikola/master aa45a68 Felix Fontein: Merge pull request #2584 from getnikola/translatable-url-pathnames...
18:24:31 -GitHub[nikola]:#nikola- [nikola] felixfontein deleted translatable-url-pathnames at 8fccc56: https://git.io/v1uhm
18:24:52 <ChrisWarrick> ralsina: ping re codacy
18:28:23 -travis-ci:#nikola- getnikola/nikola#8033 (master - aa45a68 : Felix Fontein): The build passed.
18:28:24 -travis-ci:#nikola- Change view: https://github.com/getnikola/nikola/compare/d743c673344f...aa45a6811084
18:28:24 -travis-ci:#nikola- Build details: https://travis-ci.org/getnikola/nikola/builds/182045177
18:29:45 -GitHub[nikola]:#nikola- [nikola] felixfontein pushed 3 new commits to page-range-navigation: https://git.io/v1ujJ
18:29:45 -GitHub[nikola]:#nikola- nikola/page-range-navigation b2fb036 Felix Fontein: Merge branch 'master' into page-range-navigation
18:29:45 -GitHub[nikola]:#nikola- nikola/page-range-navigation 7efd1d7 Felix Fontein: Adding explaining comments.
18:29:45 -GitHub[nikola]:#nikola- nikola/page-range-navigation 0f81c6e Felix Fontein: Updated CHANGES.txt.
18:30:10 <KwBot> [nikola] Kwpolska closed issue #2299: Add  the possibility of page range navigation https://github.com/getnikola/nikola/issues/2299
18:30:25 -GitHub[nikola]:#nikola- [nikola] Kwpolska closed pull request #2579: Added basic page range navigation. (master...page-range-navigation) https://git.io/v14Gs
18:34:04 -travis-ci:#nikola- getnikola/nikola#8034 (page-range-navigation - 0f81c6e : Felix Fontein): The build passed.
18:34:06 -travis-ci:#nikola- Change view: https://github.com/getnikola/nikola/compare/5bb86e3bcd2f...0f81c6e5ad63
18:34:06 -travis-ci:#nikola- Build details: https://travis-ci.org/getnikola/nikola/builds/182046660
18:38:09 -travis-ci:#nikola- getnikola/nikola#8036 (master - 2038779 : Chris Warrick): The build passed.
18:38:09 -travis-ci:#nikola- Change view: https://github.com/getnikola/nikola/compare/aa45a6811084...203877996e41
18:38:10 -travis-ci:#nikola- Build details: https://travis-ci.org/getnikola/nikola/builds/182046874
19:04:26 <ralsina> ChrisWarrick: on it, sorry about th delay
19:04:41 <ChrisWarrick> it's fine
19:12:37 <ralsina> ChrisWarrick: there, removed
19:16:25 <ChrisWarrick> ralsina: there’s one thing that worries me, their home page claims code coverage is pro only, so we might need to go somewhere else
19:18:15 <ChrisWarrick> we used to have coveralls set up, but that was kinda broken an the didn’t do any reporting in a visible place
19:23:08 <ralsina> there's a bunch of github-coverage-thingies
19:23:19 <ralsina> let me see what's fashionable ;-)
19:23:26 <ChrisWarrick> codacy is not a must
19:23:55 <ChrisWarrick> as long as they support python, coverage.py and perhaps have some code quality stuff (codacy is pylint-as-a-serice), we could go there
19:24:19 <ralsina> codacy has been showing really interesting things, is that all pylint? Have not used it in a while
19:24:38 <ChrisWarrick> yes, pylint but with a sane-ish config
19:25:04 <ralsina> neat
19:25:16 <ralsina> We can do two tools
19:25:31 <ChrisWarrick> or spend a day and craft a usable pylint config
19:25:57 <ChrisWarrick> last time I tried pylint was when they didn’t understand py3 at all, and it was mr. complainypants
19:26:39 <ralsina> I used to use it at work and it was just an annoyance that forced me to pepper the code with alert exceptions
19:26:52 <ralsina> Ok, so coveralls still seems to be the proper thing
19:27:23 <ChrisWarrick> pylint’s default config is unusable
19:32:42 <ChrisWarrick> then there’s codecov.io, which doesn’t look as awkward as coveralls
19:33:20 <ChrisWarrick> https://codecov.io/gh/brettcannon/cpython-ci-test https://coveralls.io/github/brettcannon/cpython-ci-test
19:44:25 <ralsina> if you like it and it's easy... I want to start working on test coverage (slowly)
19:46:12 <ChrisWarrick> switching between services takes roughly 10 minutes
19:53:50 <ChrisWarrick> as long as they don’t do stupid crap