Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Add post about deploying rails 8 to digital ocean using kamal #508

Draft
wants to merge 4 commits into
base: main
Choose a base branch
from

Conversation

chrisroos
Copy link
Member

@chrisroos chrisroos commented Feb 7, 2025

TODO: Check that I haven't broken the rendering of all existing pages by switching to the GFM renderer.
TODO: Add tests for the SnipRenderer changes if/when I'm happy that it's a sensible change

@floehopper
Copy link
Member

❤️

@chrisroos chrisroos force-pushed the add-post-about-deploying-rails-8-to-digital-ocean-using-kamal branch 2 times, most recently from e494981 to 0ae31f1 Compare February 21, 2025 17:01
In preparation for making changes to it.
So that I can take a blog post drafted in a Gist and have it display
sensibly on this site. In particular I want to use ``` code blocks
instead of the native kramdown code blocks[1].

[1]: https://kramdown.gettalong.org/syntax.html#code-blocks
Although unnecessary (because it's the default when available[1]) I've
explicitly set the `syntax_highlighter` to `rouge` as I think it makes
the code easier to reason about.

I generated the rouge-github.css stylesheet by executing `rougify style
github > app/assets/stylesheets/rouge-github.css`. This stylesheet is
automatically included in our templates by `require_tree .` in
application.css.

[1]: https://kramdown.gettalong.org/options.html#option-syntax-highlighter
@chrisroos chrisroos force-pushed the add-post-about-deploying-rails-8-to-digital-ocean-using-kamal branch from 0ae31f1 to d8e104a Compare February 21, 2025 17:07
@floehopper
Copy link
Member

@chrisroos I've opened a PR with a possible solution to the ERB problem in #516.

@chrisroos
Copy link
Member Author

chrisroos commented Feb 24, 2025 via email

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants