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

Renaming vis_default_editor, vis_type_markdown plugin folder to kebab… #419

Conversation

VachaShah
Copy link
Contributor

…-case

Signed-off-by: Vacha Shah vachshah@amazon.com

Description

This PR updates the folder names for pre-installed plugin vis-default-editor and vis-type-markdown in src/plugins to kebab-case to maintain consistency as described in Issue #322.

Testing

The following commands succeeded:

  • yarn test:jest
  • TEST_OPENSEARCH_FROM=/path/to/extracted/build/artifact yarn test:jest_integration

Issues Resolved

Part 2 of #322.
Note: This PR is one of many PRs for renaming the folder names of the pre-installed plugins.

Check List

  • New functionality includes testing.
    • All tests pass
  • New functionality has been documented.
    • New functionality has javadoc added
  • Commits are signed per the DCO using --signoff

…-case

Signed-off-by: Vacha Shah <vachshah@amazon.com>
@opensearch-ci-bot
Copy link
Collaborator

✅   DCO Check Passed a19a4ad

@kavilla
Copy link
Member

kavilla commented Jun 30, 2021

Will be verifying PRs into a branch into my fork repo. Link to PR: kavilla#21

@kavilla kavilla closed this Jun 30, 2021
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants