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

0.7.0 Cloud Plugin update - ready for review #1161

Merged
merged 8 commits into from
Jan 31, 2024

Conversation

tpmccallum
Copy link
Contributor

@tpmccallum tpmccallum commented Jan 24, 2024

Content must go through a pre-merge checklist.

Pre-Merge Content Checklist

This documentation has been checked to ensure that:

  • The title, template, and date are all set
  • Does this PR have a new menu item (anywhere in templates/*.hbs files) that points to a document .md that is set to publish in the future? If so please only publish the .md and .hbs changes in real-time (otherwise there will be a menu item pointing to a .md file that does not exist)
  • File does not use CRLF, but uses plain LF (hint: use cat -ve <filename> | grep '^M' | wc -l and expect 0 as a result)
  • Has passed bart check

$ bart check content/cloud/changelog/spin-cloud-v0-7-0.md
✅ content/cloud/changelog/spin-cloud-v0-7-0.md
$ bart check content/cloud/cloud-command-reference.md
Error rendering markdown: Error rendering "Unnamed template" line 44, col 1: Helper not defined: "tabs"
✅ content/cloud/cloud-command-reference.md

  • Has been manually tested by running in Spin/Bartholomew (hint: use PREVIEW_MODE=1 and run npm run styles to update styling)

Screenshot 2024-01-29 at 16 12 41

Screenshot 2024-01-29 at 16 11 55

  • Headings are using Title Case
  • Code blocks have the programming language set to properly highlight syntax and the proper copy directive
  • Have tested with npm run test and resolved all errors
  • Relates to an existing (potentially outdated) blog article? If so please add URL in blog to point to this content.
  • Have you searched for todo and replaced with correct information?

Signed-off-by: tpmccallum <tim.mccallum@fermyon.com>
Signed-off-by: tpmccallum <tim.mccallum@fermyon.com>
@tpmccallum tpmccallum changed the title Add first draft of 0.7.0 Cloud Plugin update 0.7.0 Cloud Plugin update - ready for review Jan 29, 2024
Copy link
Contributor

@kate-goldenring kate-goldenring left a comment

Choose a reason for hiding this comment

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

I suggested a paragraph for the change log and mentioned a couple updates to the CLI. Thanks @tpmccallum!

content/cloud/changelog/spin-cloud-v0-7-0.md Show resolved Hide resolved
content/cloud/cloud-command-reference.md Outdated Show resolved Hide resolved
content/cloud/cloud-command-reference.md Outdated Show resolved Hide resolved
content/cloud/cloud-command-reference.md Show resolved Hide resolved
Signed-off-by: Taylor Olson <taylor.olson@fermyon.com>
Signed-off-by: tpmccallum <tim.mccallum@fermyon.com>
Signed-off-by: tpmccallum <tim.mccallum@fermyon.com>
Signed-off-by: tpmccallum <tim.mccallum@fermyon.com>
Copy link
Contributor

@macolso macolso left a comment

Choose a reason for hiding this comment

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

LGTM Tim! Just one small change on the publish date.

content/cloud/changelog/spin-cloud-v0-7-0.md Outdated Show resolved Hide resolved
Signed-off-by: tpmccallum tim.mccallum@fermyon.com

Co-authored-by: MacKenzie Olson <mackenzie.olson@fermyon.com>
Signed-off-by: Timothy McCallum <tim.mccallum@fermyon.com>
Signed-off-by: Kate Goldenring <kate.goldenring@fermyon.com>
@tpmccallum tpmccallum merged commit 98ab246 into fermyon:main Jan 31, 2024
2 of 3 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

4 participants