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

Updates documentation update #962

Merged
merged 2 commits into from
Jul 1, 2020

Conversation

etungsten
Copy link
Contributor

@etungsten etungsten commented Jun 30, 2020

Issue number:
N/A

Description of changes:

Author: Erikson Tung <[email protected]>
Date:   Tue Jun 30 13:12:55 2020 -0700

    README: document more update methods
    
    Adds information about the update API and brupop to the "Updates" section of the README.

Author: Erikson Tung <[email protected]>
Date:   Tue Jun 30 12:30:43 2020 -0700

    updater documentation: fix outdated references
    
    Replaces references of `dogswatch` with `brupop`.
    Replaces references of Thar with Bottlerocket

Please look at the changes in the updater diagram using "Display the rich diff" option.

Terms of contribution:

By submitting this pull request, I agree that this contribution is dual-licensed under the terms of both the Apache License, version 2.0, and the MIT license.

Replaces references of `dogswatch` with `brupop`.
Replaces references of Thar with Bottlerocket
@etungsten etungsten changed the title Updates documentation Updates documentation update Jun 30, 2020
@etungsten etungsten requested review from bcressey, tjkirch and webern and removed request for bcressey June 30, 2020 20:36
README.md Show resolved Hide resolved
@etungsten
Copy link
Contributor Author

Addresses @webern 's comments.

@etungsten etungsten requested a review from webern June 30, 2020 22:39
Copy link
Contributor

@webern webern left a comment

Choose a reason for hiding this comment

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

🎸

README.md Outdated Show resolved Hide resolved
README.md Outdated Show resolved Hide resolved
README.md Outdated Show resolved Hide resolved
README.md Outdated Show resolved Hide resolved
README.md Outdated Show resolved Hide resolved
README.md Outdated Show resolved Hide resolved
README.md Outdated Show resolved Hide resolved
README.md Outdated Show resolved Hide resolved
README.md Outdated Show resolved Hide resolved
README.md Outdated Show resolved Hide resolved
@etungsten
Copy link
Contributor Author

Addresses @tjkirch 's comments.

@etungsten etungsten requested a review from tjkirch July 1, 2020 00:35
README.md Outdated Show resolved Hide resolved
@etungsten
Copy link
Contributor Author

Push above addresses @tjkirch 's comments and adds a simple diagram for showcasing the basic update workflow via the Update API (courtesy of @tjkirch).

README.md Outdated Show resolved Hide resolved
README.md Outdated Show resolved Hide resolved
Adds information about the update API and brupop to the "Updates" section of the README.
@etungsten
Copy link
Contributor Author

Push above renames the update api diagram file and addresses #962 (comment).

Copy link
Contributor

@zmrow zmrow left a comment

Choose a reason for hiding this comment

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

💜

@etungsten etungsten merged commit ad45b87 into bottlerocket-os:develop Jul 1, 2020
@etungsten etungsten deleted the updates-documentation branch July 1, 2020 22:33
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.

4 participants