Skip to content

build: add release notes #15621

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

Merged
merged 5 commits into from
Sep 12, 2022
Merged

Conversation

crazy-max
Copy link
Member

As discussed with @usha-mandya, carry changes from #15291 with standard release notes format while waiting for more inputs with the new one using labels.

@crazy-max crazy-max added the area/build Relates to Dockerfiles or docker build command label Sep 11, 2022
@netlify
Copy link

netlify bot commented Sep 11, 2022

Deploy Preview for docsdocker ready!

Built without sensitive environment variables

Name Link
🔨 Latest commit 7fe2679
🔍 Latest deploy log https://app.netlify.com/sites/docsdocker/deploys/631f2c2e297daa00089aa38b
😎 Deploy Preview https://deploy-preview-15621--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.

@@ -0,0 +1 @@
[{{ include.repo }}#{{ include.number }}](https://github.com/{{ include.repo }}/issues/{{ include.number }}){:target="_blank" rel="noopener" class="_"}
Copy link
Member

Choose a reason for hiding this comment

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

FWIW; I know we're using it in many places, but the "class" option may not be needed; seems just to make sure the link doesn't have a CSS class assigned (we should even consider doing the reverse, and use a special class for these links)

Copy link
Member Author

Choose a reason for hiding this comment

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

Yes many places: https://github.com/docker/docker.github.io/search?q=rel%3D%22noopener%22+class%3D%22_%22 😅. Let's track this for a follow-up.

@crazy-max crazy-max force-pushed the buildx-release-notes-std branch from 4e6da32 to 2440f83 Compare September 11, 2022 11:14
jedevc
jedevc previously approved these changes Sep 12, 2022
Copy link
Contributor

@jedevc jedevc left a comment

Choose a reason for hiding this comment

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

LGTM assuming the changes suggested by @thaJeztah 🎉

@crazy-max
Copy link
Member Author

PTAL @usha-mandya

@usha-mandya
Copy link
Member

@crazy-max Thank you SO much for the updated release notes. Left some minor comments. PTAL and let me know if you have any questions.

usha-mandya
usha-mandya previously approved these changes Sep 12, 2022
Copy link
Member

@usha-mandya usha-mandya left a comment

Choose a reason for hiding this comment

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

LGTM

@crazy-max crazy-max enabled auto-merge September 12, 2022 13:06
@crazy-max crazy-max merged commit c3a7eb5 into docker:master Sep 12, 2022
@crazy-max crazy-max deleted the buildx-release-notes-std branch September 12, 2022 13:07
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
area/build Relates to Dockerfiles or docker build command
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants