Closed adiroiban closed 10 years ago
needs-review
I have looked over the diff, it seems to have a lot in common with the parent branch. Checked README as instructed. Can we test it some other way?
approved-at 172f1a8
approved-at 172f1a8
DNS was updated and http://styleguide.chevah.com/ is not hosted on GitHub... you can run paver publish
on this branch... on Linux.
It looks like windows support is broken but I hope to fix it soon.
Thanks!
paver
, paver deps
and paver publish
all work for me. Interestingly, paver deps
gives me:
Requirement already satisfied (use --upgrade to upgrade)
so I felt the need to retry with --upgrade
… :)
approved-at 172f1a8
Problem
Styleguide pages was using an outdated hyde build system... which fails when trying to pull deps again.
Changes
Swich code to pelican which is maintained and has tests.
Beside renaming and a few small changes in templates the migration was lean as pelican is also using RST and jinja2.
Hosting was moved from Dremahost to GitHub pages. This is a big improvement since now anyone can push the changes. ... Pushing still does not work on Windows due to this bug https://github.com/davisp/ghp-import/pull/23 ... but hope the bug will be fixed soon.
build system was also update to latest paver.sh this is why the changes.
styleguide.chevah.com DNS entry was changes as a CNAME for chevah.github.io
How to test
reviewers: @alibotean @dumol depends-on: #11
This branch is based on #11 and this is why the diff is bigger... I will ask for review once parent is merged.
Check README... but this is just to let you know about the change.
This review also needs to wait for DNS to be updated.
Thanks!