-
Notifications
You must be signed in to change notification settings - Fork 867
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
[Rename] kbn-apm-config-loader to osd-apm-config-loader #78
[Rename] kbn-apm-config-loader to osd-apm-config-loader #78
Conversation
@@ -24,7 +24,7 @@ import { ApmConfiguration } from './config'; | |||
* Load the APM configuration. | |||
* | |||
* @param argv the `process.argv` arguments | |||
* @param rootDir The root directory of kibana (where the sources and the `package.json` file are) | |||
* @param rootDir The root directory of opensearchDashboards (where the sources and the `package.json` file are) |
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.
This should be opensearch-dashboards.
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.
LGTM
Signed-off-by: Kawika Avilla <kavilla414@gmail.com>
Signed-off-by: Kawika Avilla <kavilla414@gmail.com>
2b89127
to
a46a377
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.
LGTM !!
* [Rename] kbn-apm-config-loader to osd-apm-onfig-loader Signed-off-by: Kawika Avilla <kavilla414@gmail.com> * Handle PR comments Signed-off-by: Kawika Avilla <kavilla414@gmail.com>
* [Rename] kbn-apm-config-loader to osd-apm-onfig-loader Signed-off-by: Kawika Avilla <kavilla414@gmail.com> * Handle PR comments Signed-off-by: Kawika Avilla <kavilla414@gmail.com>
* [Rename] kbn-apm-config-loader to osd-apm-onfig-loader Signed-off-by: Kawika Avilla <kavilla414@gmail.com> * Handle PR comments Signed-off-by: Kawika Avilla <kavilla414@gmail.com>
…ject#78) refactor: simplified ACL transformPermission function --------- Signed-off-by: Yulong Ruan <ruanyl@amazon.com>
…ject#78) refactor: simplified ACL transformPermission function --------- Signed-off-by: Yulong Ruan <ruanyl@amazon.com>
…ject#78) refactor: simplified ACL transformPermission function --------- Signed-off-by: Yulong Ruan <ruanyl@amazon.com>
Signed-off-by: Kawika Avilla kavilla414@gmail.com
Issue #, if available:
#37
Description of changes:
Rename folders, files, and variables to osd-apm-config-loader
By submitting this pull request, I confirm that you can use, modify, copy, and redistribute this contribution, under the terms of your choice.