Skip to content

Conversation

@aevesdocker
Copy link
Contributor

No description provided.

@aevesdocker aevesdocker requested a review from dvdksn September 5, 2022 13:22
@netlify
Copy link

netlify bot commented Sep 5, 2022

Deploy Preview for docsdocker ready!

Built without sensitive environment variables

Name Link
🔨 Latest commit 0d2c194
🔍 Latest deploy log https://app.netlify.com/sites/docsdocker/deploys/6315f7e339ad20000842ac55
😎 Deploy Preview https://deploy-preview-15565--docsdocker.netlify.app
📱 Preview on mobile
Toggle QR Code...

QR Code

Use your smartphone camera to open QR code link.

To edit notification comments on pull requests, go to your Netlify site settings.

@aevesdocker aevesdocker merged commit 1c273c4 into docker:master Sep 5, 2022
## Copyright and license

Copyright 2013-2022 Docker, inc, released under the Apache 2.0 license.
We have created a ['Contribute' section](/contribute/overview.md) in our docs which guides you through the process of contributing to Docker documentation.
Copy link
Member

@crazy-max crazy-max Sep 5, 2022

Choose a reason for hiding this comment

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

It might have been better to put the full link for contributing overview otherwise contributors will see internal markdown of GitHub with frontmatter stuff and bad formatting:

image

Smth like:

['Contribute' section]({{ site.docs_url }}/contribute/overview/)

This way contributors from GitHub or docs website will have the same layout.

Copy link
Member

@crazy-max crazy-max Sep 5, 2022

Choose a reason for hiding this comment

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

Oh actually it will not work as GitHub does not handle liquid syntax in their markdown 😣

Copy link
Contributor Author

Choose a reason for hiding this comment

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

Ah, thanks for spotting this Kevin. Have amended in #15567

aevesdocker added a commit that referenced this pull request Sep 5, 2022
* Update mac-install.md (#15555)

Typo in spelling of subscription. `subscri[tion` -> `subscription`

* Addition of a 'Contribute' section with our Style Guide (#15350)

* SG test.

* moved some content around

* editing

* broken links fix

* formatting changes

* fixes

* fixes

* edits after review

* fix broken links

* broken link fix

* a few more edits

* removed contributing page

* Revert "removed contributing page"

This reverts commit 5822cd5.

* add redirect

* Install page improvement based on feedback (scenario2) (#15494)

* Install page improvement based on users and PM feedback
* Exploring a second scenario deriving from additional feedback
* Review feedback

* contribute edit (#15565)

Co-authored-by: Clint Joseph <clintjoseph1992@gmail.com>
Co-authored-by: Dockertopia <103652162+dockertopia@users.noreply.github.com>
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.

3 participants