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

[GUIDE] Add document_lifecycle guide #290

Merged

Conversation

zethuman
Copy link
Contributor

@zethuman zethuman commented Apr 10, 2023

Description

Add document_lifecycle guide.

Issues Resolved

Closes [#276, #279].

By submitting this pull request, I confirm that my contribution is made under the terms of the Apache 2.0 license.
For more information on following Developer Certificate of Origin and signing off your commits, please check here.

@zethuman zethuman changed the title Add document_lifecycle guide [GUIDE] Add document_lifecycle guide Apr 10, 2023
@dblock
Copy link
Member

dblock commented Apr 12, 2023

@zethuman I think you'll have easier time doing this:

git checkout main
git pull upsteam main
git checkout -b feature/guides/all
git merge --squash feature/guides/document_lifecycle
... repeat with every one of your feature branches
... edit/update CHANGELOG
git commit -s -m "Created user guides."
git push origin feature/guides/all

In the end you'll have one nice PR for all the guides. I think it's faster than fixing DCO in every PR. WDYT?

@zethuman zethuman force-pushed the feature/guides/document_lifecycle branch from f11952d to fa32f4d Compare April 12, 2023 17:24
zethuman and others added 5 commits April 12, 2023 23:24
* Add linter for markdown files

Signed-off-by: Rakhat Zhuman <zhumanrakhat01@gmail.com>

* Updated CHANGELOG.md

Signed-off-by: Rakhat Zhuman <zhumanrakhat01@gmail.com>

* Add make-script and update DEVELOPER_GUIDE.md

Signed-off-by: Rakhat Zhuman <zhumanrakhat01@gmail.com>

* Updated linter options

Signed-off-by: Rakhat Zhuman <zhumanrakhat01@gmail.com>

* Fixed mmarkdown files format

Signed-off-by: Rakhat Zhuman <zhumanrakhat01@gmail.com>

---------

Signed-off-by: Rakhat Zhuman <zhumanrakhat01@gmail.com>
…t#272)

Signed-off-by: dblock <dblock@amazon.com>
Signed-off-by: Rakhat Zhuman <zhumanrakhat01@gmail.com>
Signed-off-by: Rakhat Zhuman <zhumanrakhat01@gmail.com>
Signed-off-by: Rakhat Zhuman <zhumanrakhat01@gmail.com>
…ensearch-project#286)

Signed-off-by: Jakob Hahn <jakob.hahn@hetzner.com>
Signed-off-by: Rakhat Zhuman <zhumanrakhat01@gmail.com>
@zethuman zethuman force-pushed the feature/guides/document_lifecycle branch from fa32f4d to d3a8912 Compare April 12, 2023 17:24
@zethuman
Copy link
Contributor Author

Yes, I thought about it, but it's probably more a matter of principle...

@Jakob3xD
Copy link
Collaborator

AFAIK this is the only other Guide where the go code is not indented correctly.

Signed-off-by: Rakhat Zhuman <zhumanrakhat01@gmail.com>
@zethuman zethuman force-pushed the feature/guides/document_lifecycle branch from 3004ed4 to d586aa2 Compare April 14, 2023 15:25
@zethuman
Copy link
Contributor Author

@Jakob3xD add indented go code

Signed-off-by: Rakhat Zhuman <zhumanrakhat01@gmail.com>
@zethuman zethuman force-pushed the feature/guides/document_lifecycle branch from bc6898b to f34da71 Compare April 14, 2023 15:31
@dblock dblock merged commit a6b20de into opensearch-project:main Apr 14, 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.

3 participants