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

Refactor LifecycleComponent package path #377

Merged
merged 1 commit into from
Aug 3, 2023

Conversation

heemin32
Copy link
Collaborator

@heemin32 heemin32 commented Aug 3, 2023

Description

Refactor LifecycleComponent package path

Issues Resolved

N/A

Check List

  • Commits are signed per the DCO using --signoff

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.

@heemin32
Copy link
Collaborator Author

heemin32 commented Aug 3, 2023

Integration test is failing because the change in core is not released to maven repo yet.

@codecov
Copy link

codecov bot commented Aug 3, 2023

Codecov Report

Merging #377 (235b810) into main (e1f41bb) will decrease coverage by 0.04%.
The diff coverage is n/a.

@@             Coverage Diff              @@
##               main     #377      +/-   ##
============================================
- Coverage     88.67%   88.63%   -0.04%     
+ Complexity      753      752       -1     
============================================
  Files            92       92              
  Lines          2711     2711              
  Branches        220      220              
============================================
- Hits           2404     2403       -1     
  Misses          228      228              
- Partials         79       80       +1     
Files Changed Coverage Δ
...rch/geospatial/ip2geo/listener/Ip2GeoListener.java 81.57% <ø> (ø)
...opensearch/geospatial/plugin/GeospatialPlugin.java 90.69% <ø> (ø)

... and 1 file with indirect coverage changes

@heemin32 heemin32 merged commit d509bfc into opensearch-project:main Aug 3, 2023
@heemin32 heemin32 deleted the patch branch August 3, 2023 21:05
@heemin32 heemin32 added the backport 2.x Backport PR to 2.x branch label Aug 16, 2023
opensearch-trigger-bot bot pushed a commit that referenced this pull request Aug 16, 2023
Signed-off-by: Heemin Kim <[email protected]>
(cherry picked from commit d509bfc)
heemin32 added a commit to heemin32/geospatial that referenced this pull request Aug 17, 2023
heemin32 added a commit that referenced this pull request Aug 17, 2023
)

* Switch from XContentType to MediaType to fix compilation errors (#366)

Signed-off-by: Heemin Kim <[email protected]>

* Refactor XContentFactory to MediaTypeRegistry (#374)

Signed-off-by: Heemin Kim <[email protected]>

* Refactor LifecycleComponent package path (#377)

Signed-off-by: Heemin Kim <[email protected]>

* [Refactor] Strings utility methods to core library (#379)

Signed-off-by: Heemin Kim <[email protected]>

* Fixed compilation errors after refactoring in core foundation classes (#380)

Signed-off-by: Heemin Kim <[email protected]>

---------

Signed-off-by: Heemin Kim <[email protected]>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
backport 2.x Backport PR to 2.x branch
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants