Skip to content

Chapter 5 octupdates #15353

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 108 commits into from
Jan 28, 2020
Merged

Chapter 5 octupdates #15353

merged 108 commits into from
Jan 28, 2020

Conversation

robvet
Copy link
Contributor

@robvet robvet commented Oct 22, 2019

Cloud-Native Book
Chapter 5 (data) Updates

@mairaw
Copy link
Contributor

mairaw commented Oct 22, 2019

This replaces #15343, right?

robvet and others added 4 commits January 22, 2020 19:21
Copy link
Member

@nishanil nishanil left a comment

Choose a reason for hiding this comment

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

LGTM. @mairaw As discussed, as long as we are good with the third-party logos, we can merge as-is. If not, we can remove that image and merge in.

Copy link
Contributor

@mairaw mairaw left a comment

Choose a reason for hiding this comment

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

A few extra comments

Co-Authored-By: Maira Wenzel <mairaw@microsoft.com>
robvet and others added 7 commits January 23, 2020 19:16
Co-Authored-By: Maira Wenzel <mairaw@microsoft.com>
Co-Authored-By: Maira Wenzel <mairaw@microsoft.com>
Co-Authored-By: Maira Wenzel <mairaw@microsoft.com>
Co-Authored-By: Maira Wenzel <mairaw@microsoft.com>
Co-Authored-By: Maira Wenzel <mairaw@microsoft.com>
Co-Authored-By: Maira Wenzel <mairaw@microsoft.com>
@robvet
Copy link
Contributor Author

robvet commented Jan 24, 2020 via email

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

5 participants