Add Forum link in SQL plugin README.md by cwillum · Pull Request #809 · opensearch-project/sql · GitHub
Skip to content

Add Forum link in SQL plugin README.md#809

Merged
MaxKsyunz merged 3 commits into
opensearch-project:mainfrom
cwillum:plugin-README-links-SQL
Sep 12, 2022
Merged

Add Forum link in SQL plugin README.md#809
MaxKsyunz merged 3 commits into
opensearch-project:mainfrom
cwillum:plugin-README-links-SQL

Conversation

@cwillum

@cwillum cwillum commented Sep 8, 2022

Copy link
Copy Markdown
Contributor

Signed-off-by: cwillum cwmmoore@amazon.com

Description

Added Forum header section that includes verbiage and a link to the OpenSearch Forum page for the SQL plugin.

Issues Resolved

This more closely corresponds to the information found in the README.md file in all of the other plugins bundled with OpenSearch.

This fixes the issue for the SQL plugin in #921.

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.

Signed-off-by: cwillum <cwmmoore@amazon.com>
Comment thread README.md Outdated
Signed-off-by: cwillum <cwmmoore@amazon.com>
@codecov-commenter

codecov-commenter commented Sep 8, 2022

Copy link
Copy Markdown

dai-chen
dai-chen previously approved these changes Sep 8, 2022

@dai-chen dai-chen left a comment

Copy link
Copy Markdown
Collaborator

Choose a reason for hiding this comment

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

Thanks for the changes!

@dai-chen dai-chen added the documentation Improvements or additions to documentation label Sep 8, 2022
@cwillum

cwillum commented Sep 9, 2022

Copy link
Copy Markdown
Contributor Author

Comment thread README.md
Comment thread README.md Outdated
MaxKsyunz
MaxKsyunz previously approved these changes Sep 10, 2022
Signed-off-by: cwillum <cwmmoore@amazon.com>
@cwillum cwillum dismissed stale reviews from MaxKsyunz and dai-chen via 17783a6 September 12, 2022 17:37
@MaxKsyunz MaxKsyunz merged commit ac011ce into opensearch-project:main Sep 12, 2022
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

documentation Improvements or additions to documentation

Projects

None yet

Development

Successfully merging this pull request may close these issues.

6 participants