Skip to content

Conversation

@cnunciato
Copy link
Contributor

This is still a WIP, but is in a good-enough spot to merge into fusion in order to keep @chrsmith unblocked on the blog work.

cnunciato added 6 commits June 7, 2019 15:13
Signed-off-by: Christian Nunciato <c@nunciato.org>
Signed-off-by: Christian Nunciato <c@nunciato.org>
Signed-off-by: Christian Nunciato <c@nunciato.org>
Signed-off-by: Christian Nunciato <c@nunciato.org>
Signed-off-by: Christian Nunciato <c@nunciato.org>
Signed-off-by: Christian Nunciato <c@nunciato.org>
@cnunciato cnunciato requested review from chrsmith and justinvp June 7, 2019 22:21
cnunciato and others added 12 commits June 7, 2019 15:40
Signed-off-by: Christian Nunciato <c@nunciato.org>
Signed-off-by: Christian Nunciato <c@nunciato.org>
Signed-off-by: Christian Nunciato <c@nunciato.org>
Signed-off-by: Christian Nunciato <c@nunciato.org>
Signed-off-by: Christian Nunciato <c@nunciato.org>
Signed-off-by: Christian Nunciato <c@nunciato.org>
Signed-off-by: Christian Nunciato <c@nunciato.org>
Signed-off-by: Christian Nunciato <c@nunciato.org>
Signed-off-by: Christian Nunciato <c@nunciato.org>
Signed-off-by: Christian Nunciato <c@nunciato.org>
@cnunciato
Copy link
Contributor Author

@chrsmith Can we merge this into fusion and then continue working from there, after we merge your other PR into this branch?

@justinvp
Copy link
Member

justinvp commented Jun 9, 2019

What's the benefit of the extra assets dir in static/assets/images? I know that's how it is in the www repo, but looking at some other Hugo sites listed at https://gohugo.io/showcase/, they all seem to put things directly in static, e.g. static/images, static/fonts, etc., not static/assets/images, static/assets/fonts, etc.

Co-Authored-By: Justin Van Patten <jvp@justinvp.com>
Copy link
Member

@justinvp justinvp left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

I did a quick scan of all the content, but did not look at it in detail. Other than my question about the assets dir (which we can deal with or not subsequently), looks good to me to merge into fusion.

weight: 2

hero_title: "Acceptable Use Policy"
url: "/acceptable-use"
Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

As a subsequent exercise, I wonder if we should audit all the uses of url: and consider just placing these files where they should be, e.g. instead of having this in /content/legal/acceptable-use.md with a URL that makes it /acceptable-use, delete the url: front matter and just place the file directly in /content/acceptable-use.md

@@ -0,0 +1,15 @@
---
title: "Cloud Delivery Service"
date: 2018-06-01T15:10:06-07:00
Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

As a subsequent exercise, we should consider deleting the dates from most of these content files, as I think they're just "dead code", except for blog posts where date is used.

title: "Cloud Delivery Service"
date: 2018-06-01T15:10:06-07:00
layout: "cloud-delivery-service"
draft: true
Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

We should also do an audit of draft: at some point. Are these pages not being published currently? If not, can we remove? If they are we should delete the draft: from the front matter.

Copy link
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

I would, yeah. No sense keeping around what clearly isn't in use!

</div>
<div class="anim_textbox code_display_textbox">
<div class="code-display">
<pre><code class="javascript hljs">import * as k8s from "@pulumi/kubernetes";
Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Just a note: In the short term, we'll want to get https://highlightjs.org setup so these code snippets are syntax highlighted. The blog also uses highlight.js for highlighting code snippets, so it'd also highlight existing code snippets in blog posts (until they've been converted to markdown and get Hugo's built-in Chroma-based highlighting).

Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

cc: @chrsmith as he mentioned he's converting the blog posts over to use Hugo's highlight shortcode. Although, I suggest we standardize on Code Fences long-term for most code snippets (unless we need something more specialized like line numbers and line highlighting), as that's what's used everywhere else in existing docs content and in API doc comments in package repos.

@cnunciato
Copy link
Contributor Author

What's the benefit of the extra assets dir in static/assets/images?

None, really -- I'm just as happy having things at /images, etc. I'll make that change.

Signed-off-by: Christian Nunciato <c@nunciato.org>
Signed-off-by: Christian Nunciato <c@nunciato.org>
@cnunciato cnunciato merged commit 351af52 into fusion Jun 10, 2019
@pulumi-bot pulumi-bot deleted the cnunciato/add-www branch June 10, 2019 16:47
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.

4 participants