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

[Rename] all elastic github references except for issues and PRs (#37) #200

Merged
merged 2 commits into from
Mar 19, 2021
Merged

Conversation

boktorbb
Copy link
Contributor

Signed-off-by: Bishoy Boktor boktorbb@amazon.com

@odfe-release-bot
Copy link

✅   DCO Check Passed

Signed-off-by: Bishoy Boktor <boktorbb@amazon.com>
@odfe-release-bot
Copy link

✅   DCO Check Passed

CONTRIBUTING.md Outdated Show resolved Hide resolved
@@ -272,7 +272,7 @@ The tests cover:
- authenticated / non-authenticated user access (when applicable)

```typescript
// TODO after https://github.com/elastic/opensearch-dashboards/pull/53208/
// TODO after https://github.com/elastic/kibana/pull/53208/
Copy link
Member

Choose a reason for hiding this comment

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

good catch

@kavilla
Copy link
Member

kavilla commented Mar 19, 2021

Mostly good just that conflict was committed

Signed-off-by: Bishoy Boktor <boktorbb@amazon.com>
Copy link
Member

@kavilla kavilla left a comment

Choose a reason for hiding this comment

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

LGTM thanks!

@odfe-release-bot
Copy link

✅   DCO Check Passed

@boktorbb boktorbb merged commit 72d9ffb into opensearch-project:dashboards Mar 19, 2021
Copy link
Contributor

@mihirsoni mihirsoni left a comment

Choose a reason for hiding this comment

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

LGTM

kavilla referenced this pull request in kavilla/OpenSearch-Dashboards Mar 19, 2021
…) (#200)

* [Rename] all elastic github references except for issues and PRs (#37)

Signed-off-by: Bishoy Boktor <boktorbb@amazon.com>

* Address PR comments

Signed-off-by: Bishoy Boktor <boktorbb@amazon.com>
kavilla pushed a commit that referenced this pull request Mar 20, 2021
#200)

* [Rename] all elastic github references except for issues and PRs (#37)

Signed-off-by: Bishoy Boktor <boktorbb@amazon.com>

* Address PR comments

Signed-off-by: Bishoy Boktor <boktorbb@amazon.com>
mihirsoni pushed a commit that referenced this pull request Mar 20, 2021
#200)

* [Rename] all elastic github references except for issues and PRs (#37)

Signed-off-by: Bishoy Boktor <boktorbb@amazon.com>

* Address PR comments

Signed-off-by: Bishoy Boktor <boktorbb@amazon.com>
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.

5 participants