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

Update docs with new changes; Add info about new kanister func #211

Merged
merged 8 commits into from
Aug 21, 2019

Conversation

SupriyaKasten
Copy link
Contributor

Change Overview

Update docs with; add info about new kanister func

Pull request type

Please check the type of change your PR introduces:

  • Work in Progress
  • Refactoring (no functional changes, no api changes)
  • Trival/Minor
  • Bugfix
  • Feature
  • Documentation

@tdmanv
Copy link
Contributor

tdmanv commented Aug 16, 2019

./functions.rst:612: D001 Line too long

Copy link
Contributor

@tdmanv tdmanv left a comment

Choose a reason for hiding this comment

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

I don't understand the wording of the PR title - please make it a bit clearer before merging

@SupriyaKasten SupriyaKasten changed the title Update docs with; add info about new kanister func Update docs with new changes; Add info about new kanister func Aug 16, 2019
@SupriyaKasten
Copy link
Contributor Author

@tdmanv my bad!

@@ -149,6 +152,7 @@ as follows:
ConfigMaps map[string]ObjectReference `json:"configMaps"`
Secrets map[string]ObjectReference `json:"secrets"`
Profile *ObjectReference `json:"profile"`
Options map[string]string `json:"options"`
Copy link
Contributor

Choose a reason for hiding this comment

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

[minor] Please check formatting here. Looks like italics is enabled.

Copy link
Contributor

@pavannd1 pavannd1 Aug 21, 2019

Choose a reason for hiding this comment

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

@SupriyaKasten I think this is because of *ObjectReference. * enabled italics. Let's move your change above Profile for now.

Copy link
Contributor Author

Choose a reason for hiding this comment

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

Sure!

docs/functions.rst Outdated Show resolved Hide resolved
docs/functions.rst Outdated Show resolved Hide resolved
docs/functions.rst Outdated Show resolved Hide resolved
docs/functions.rst Outdated Show resolved Hide resolved
docs/functions.rst Outdated Show resolved Hide resolved
docs/functions.rst Outdated Show resolved Hide resolved
docs/spelling_wordlist.txt Outdated Show resolved Hide resolved
Copy link
Contributor

@pavannd1 pavannd1 left a comment

Choose a reason for hiding this comment

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

Minor comments. Good to go otherwise.

docs/functions.rst Outdated Show resolved Hide resolved
SupriyaKasten and others added 2 commits August 21, 2019 14:50
Co-Authored-By: Pavan Navarathna Devaraj <pavan@kasten.io>
@SupriyaKasten SupriyaKasten merged commit a418c71 into master Aug 21, 2019
@SupriyaKasten SupriyaKasten deleted the docs-update branch August 21, 2019 22:07
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.

None yet

3 participants