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] Renames elastic-safer-lodash-set dependency #252

Merged
merged 1 commit into from
Apr 8, 2021
Merged

[Rename] Renames elastic-safer-lodash-set dependency #252

merged 1 commit into from
Apr 8, 2021

Conversation

boktorbb
Copy link
Contributor

@boktorbb boktorbb commented Apr 6, 2021

Description

Renames elastic-safer-lodash-set --> opensearch-safer-lodash-set

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

@boktorbb boktorbb requested review from ananzh, kavilla and mihirsoni April 6, 2021 22:12
@boktorbb boktorbb added this to the Beta release milestone Apr 6, 2021
@odfe-release-bot
Copy link

✅   DCO Check Passed 17ab2b6

Renames the elastic-safer-lodash-set to opensearch-safer-lodash-set and
updated precommit hook rules and eslintignore since this package file
names need to match upstream lodash

Signed-off-by: Bishoy Boktor <boktorbb@amazon.com>
@boktorbb boktorbb changed the title [Rename] Renames upstream elastic dependency packages [Rename] Renames elastic-safer-lodash-set dependency Apr 7, 2021
@odfe-release-bot
Copy link

✅   DCO Check Passed bf269d7

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 !! Still concerned on why this files are deleted / added :(

@kavilla
Copy link
Member

kavilla commented Apr 7, 2021

Still references to the old package name in the README, did we want to update those? Also did we want to update the package.json values? I am unable to build this PR. Should I be pulling down all the PRs for building?

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.

Also, in .eslintrc.js we tell people to use this package instead of the original lodash in when they try to use lodash.set, lodash.setWith. Did we want to update that as well with this PR?

@boktorbb
Copy link
Contributor Author

boktorbb commented Apr 7, 2021

Still references to the old package name in the README, did we want to update those? Also did we want to update the package.json values? I am unable to build this PR. Should I be pulling down all the PRs for building?

Yes, can you pull all 3 PRs down

@boktorbb
Copy link
Contributor Author

boktorbb commented Apr 7, 2021

Still references to the old package name in the README, did we want to update those? Also did we want to update the package.json values? I am unable to build this PR. Should I be pulling down all the PRs for building?

I was directed to only rename the top level dir and nothing else

@kavilla
Copy link
Member

kavilla commented Apr 7, 2021

we are told to keep things mostly the same, so after cherry-picking all the commits, everything resolved so my comments can be resolved.

@kavilla kavilla self-requested a review April 7, 2021 21:54
@boktorbb boktorbb merged commit f5cc039 into opensearch-project:main Apr 8, 2021
boktorbb added a commit that referenced this pull request Apr 8, 2021
Renames the elastic-safer-lodash-set to opensearch-safer-lodash-set and
updated precommit hook rules and eslintignore since this package file
names need to match upstream lodash

Signed-off-by: Bishoy Boktor <boktorbb@amazon.com>
@joshuarrrr joshuarrrr mentioned this pull request Jun 23, 2022
7 tasks
wanglam added a commit to wanglam/OpenSearch-Dashboards that referenced this pull request Feb 27, 2024
SuZhou-Joe pushed a commit to SuZhou-Joe/OpenSearch-Dashboards that referenced this pull request Feb 28, 2024
…opensearch-project#253)

(cherry picked from commit 734c890)

Signed-off-by: Lin Wang <wonglam@amazon.com>
Signed-off-by: github-actions[bot] <github-actions[bot]@users.noreply.github.com>
Co-authored-by: github-actions[bot] <github-actions[bot]@users.noreply.github.com>
Hailong-am pushed a commit to Hailong-am/OpenSearch-Dashboards that referenced this pull request Mar 7, 2024
…n management and SavedObject management as standalone app, retire dashboard management (opensearch-project#283)

* Revert "Register Advance Settings, Data Source management,Index Pattern management and SavedObject management as standalone app, retire dashboard management (opensearch-project#208)"

This reverts commit 5f09f84.

* Revert "feat: add unit test for mountWrapper (opensearch-project#223)"

This reverts commit a70ce71.

* Revert "fix: cypress tests checkout workspace branch (opensearch-project#252)"

This reverts commit 734c890.

* run cypress with ftr main

Signed-off-by: Yulong Ruan <ruanyl@amazon.com>

---------

Signed-off-by: Yulong Ruan <ruanyl@amazon.com>
SuZhou-Joe pushed a commit to SuZhou-Joe/OpenSearch-Dashboards that referenced this pull request Mar 18, 2024
SuZhou-Joe pushed a commit to SuZhou-Joe/OpenSearch-Dashboards that referenced this pull request Mar 18, 2024
…n management and SavedObject management as standalone app, retire dashboard management (opensearch-project#283)

* Revert "Register Advance Settings, Data Source management,Index Pattern management and SavedObject management as standalone app, retire dashboard management (opensearch-project#208)"

This reverts commit 5f09f84.

* Revert "feat: add unit test for mountWrapper (opensearch-project#223)"

This reverts commit a70ce71.

* Revert "fix: cypress tests checkout workspace branch (opensearch-project#252)"

This reverts commit 734c890.

* run cypress with ftr main

Signed-off-by: Yulong Ruan <ruanyl@amazon.com>

---------

Signed-off-by: Yulong Ruan <ruanyl@amazon.com>
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