-
Notifications
You must be signed in to change notification settings - Fork 58
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
Update base-images.md #323
Conversation
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Let me do a quick copyedit of this page. Will do right now and unblock after I commit it.
cb644eb
to
5e8629d
Compare
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
@pankajroark I tried my best to clarify and shorten the docs. Please let me know if I left out any key details.
I also re-organized this as a "contributor" doc because it only seems relevant to people contributing to Truss, not everyday users. If that's incorrect, I'll put it back with the main docs.
@pankajroark is this good to merge? |
@philipkiely-baseten this looks good. Can you just resolve the conflicts and we'll merge it? |
4277b25
to
3e0f97f
Compare
Cool, rebased and merged! |
No description provided.