cloudamqp / amqp-client.cr

An AMQP 0-9-1 client for Crystal
https://cloudamqp.github.io/amqp-client.cr/
MIT License
69 stars 10 forks source link

Remove and automate docs generation #17

Closed Blacksmoke16 closed 4 years ago

Blacksmoke16 commented 4 years ago

Example build: https://github.com/Blacksmoke16/amqp-client.cr/runs/1126200835?check_suite_focus=true Example output: https://github.com/Blacksmoke16/amqp-client.cr/tree/gh-pages Example docs: https://blacksmoke16.github.io/amqp-client.cr/

NOTE: Currently Crystal docker images are a bit dated. Could update the image/remove the need to manually install rsync once there are more proper images resulting from https://github.com/crystal-lang/distribution-scripts/pull/68.

dentarg commented 4 years ago

@carlhoerberg WDYT?

carlhoerberg commented 4 years ago

I like that the docs are built for amq-protocol too. But can we get rid if the container and install crystal via snap instead? Less to download, less layers

Blacksmoke16 commented 4 years ago

@carlhoerberg Probably, but would it be worth it? The workflow already only takes 30s with the container being 9s of that.

carlhoerberg commented 4 years ago

Thanks! updated the website to use the gh-pages branch