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

docs: slsa editorial fixes #3

Merged

Conversation

dvdksn
Copy link

@dvdksn dvdksn commented Dec 22, 2022

Updates moby#3375

Signed-off-by: David Karlsson david.karlsson@docker.com

  • Editorial improvements
  • Added more headings
  • Added clearer indication for which mode the attestation fields are included in

Also forked the field definitions into a separate page. My intention here was to
make this content reusable by the Docker docs repository downstream. In addition
to moving the text to a separate file, I had to remove a couple of references to
buildctl, to make this page client-agnostic.

Signed-off-by: David Karlsson <david.karlsson@docker.com>
Signed-off-by: David Karlsson <david.karlsson@docker.com>
@dvdksn dvdksn force-pushed the docs/slsa-editorial branch from 1a2277a to c1d5282 Compare December 22, 2022 10:11
Copy link
Owner

@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.

Couple minor nits, the editorial fixes look good to me.

I think splitting up into multiple files should be good, PTAL @tonistiigi.

docs/slsa.md Outdated Show resolved Hide resolved
docs/slsa.md Outdated Show resolved Hide resolved
@jedevc jedevc merged commit fa99051 into jedevc:attestations-provenance-docs Jan 4, 2023
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.

2 participants