-
Notifications
You must be signed in to change notification settings - Fork 229
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
Adds basic blog functionality #339
Conversation
✅ Deploy Preview for slsa ready!
To edit notification comments on pull requests, go to your Netlify site settings. |
Thanks so much! @MarkLodato did you want to review this or are you happy for someone else to do so? |
@TomHennen feel free to review as well. Also, i think some minor layout things can be improved (e.g. spacing before each blog title, split row colors e.g. on overview page, etc), this can be come in next CL. |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Thanks, Jory! I think this is great.
Note that the spacing between paragraph is bad due to #335, if anyone is willing to fix that in a separate PR.
…Burson <jburson@linuxfoundation.org>
Closes #309
Adds:
Note:
I am not a designer 😂 but see attached screenshots; need to nuke _posts before merging (unless someone wants to merge with an inaugural blog post!)
Signed-off-by: Jory Burson jburson@linuxfoundation.org