Skip to content

Add missing code-hints, and minor markdown edits - #13352

Merged
usha-mandya merged 1 commit into
docker:masterfrom
thaJeztah:add_missing_codehints
Aug 16, 2021
Merged

Add missing code-hints, and minor markdown edits#13352
usha-mandya merged 1 commit into
docker:masterfrom
thaJeztah:add_missing_codehints

Conversation

@thaJeztah

@thaJeztahthaJeztah commented Aug 14, 2021

Copy link
Copy Markdown
Member

foillow-up to #13299

  • Add missing code-hints (console, yaml)
  • Consistently add an empty line after code-blocks
  • Combine some examples where the output and the command were put in separate blocks. With the "console" code-hint, this is no longer nescessary.
  • fix indentation in cloud/ecs-integration.md, which caused the numbered-list to be interrupted.

@thaJeztah
thaJeztahforce-pushed the add_missing_codehints branch from 13221c4 to 5d9e941CompareAugust 14, 2021 12:48
@netlify

This comment has been minimized.

@thaJeztah
thaJeztahforce-pushed the add_missing_codehints branch 2 times, most recently from 6a216a6 to 691aa55CompareAugust 14, 2021 12:55
@netlify

This comment has been minimized.

@thaJeztah
thaJeztahforce-pushed the add_missing_codehints branch from 691aa55 to b3750edCompareAugust 14, 2021 12:57
@netlify

netlifyBot commented Aug 14, 2021

Copy link
Copy Markdown

✔️ Deploy Preview for docsdocker ready!

🔨 Explore the source changes: 691aa55

🔍 Inspect the deploy log: https://app.netlify.com/sites/docsdocker/deploys/6117bd2644e608000878f4a2

😎 Browse the preview: https://deploy-preview-13352--docsdocker.netlify.app

@netlify

netlifyBot commented Aug 14, 2021

Copy link
Copy Markdown

✔️ Deploy Preview for docsdocker ready!

🔨 Explore the source changes: 1583678

🔍 Inspect the deploy log: https://app.netlify.com/sites/docsdocker/deploys/611a45147e05ef00077295bd

😎 Browse the preview: https://deploy-preview-13352--docsdocker.netlify.app

@thaJeztah

Copy link
Copy Markdown
MemberAuthor

@usha-mandya@StefanScherer PTAL

StefanScherer
StefanScherer previously approved these changes Aug 16, 2021

@StefanSchererStefanScherer left a comment

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

LGTM 🧹

@thaJeztah

Copy link
Copy Markdown
MemberAuthor

FWIW: If we are good with #13353, I think rebasing this after that one will be easier (git should in most cases automatically pick up the renamed files)

- Add missing code-hints (console, yaml)
- Consistently add an empty line after code-blocks
- Combine some examples where the output and the command were
put in separate blocks. With the "console" code-hint, this
is no longer nescessary.
- fix indentation in cloud/ecs-integration.md, which caused the
numbered-list to be interrupted.
Signed-off-by: Sebastiaan van Stijn <github@gone.nl>
@thaJeztah

Copy link
Copy Markdown
MemberAuthor

Rebased, as #13353 was merged 👍

@usha-mandyausha-mandya left a comment

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

LGTM

@StefanSchererStefanScherer left a comment

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

LGTM

@usha-mandya
usha-mandya merged commit 00bbe69 into docker:masterAug 16, 2021
@thaJeztah
thaJeztah deleted the add_missing_codehints branch August 16, 2021 11:16
Sign up for freeto 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.

3 participants

@thaJeztah@StefanScherer@usha-mandya