Skip to content

Add group = 'org.opensearch' to build.gradle#111

Merged
junqiu-lei merged 2 commits intoopensearch-project:mainfrom
prudhvigodithi:main
Aug 3, 2022
Merged

Add group = 'org.opensearch' to build.gradle#111
junqiu-lei merged 2 commits intoopensearch-project:mainfrom
prudhvigodithi:main

Conversation

@prudhvigodithi
Copy link
Copy Markdown
Member

@prudhvigodithi prudhvigodithi commented Aug 3, 2022

Signed-off-by: prudhvigodithi pgodithi@amazon.com

Description

Issue: fixes the following error

> Task :publishNebulaPublicationToMavenLocal FAILED
Execution optimizations have been disabled for task ':publishNebulaPublicationToMavenLocal' to ensure correctness due to the following reasons:
  - Gradle detected a problem with the following location: '/Users/balasvij/Documents/WORK/geospatial/build/distributions/opensearch-geospatial-2.2.0.0-SNAPSHOT.pom'. Reason: Task ':publishNebulaPublicationToMavenLocal' uses this output of task ':generatePomFileForPluginZipPublication' without declaring an explicit or implicit dependency. This can lead to incorrect results being produced, depending on what order the tasks are executed. Please refer to https://docs.gradle.org/7.4.1/userguide/validation_problems.html#implicit_dependency for more details about this problem.

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.

Signed-off-by: prudhvigodithi <pgodithi@amazon.com>
@prudhvigodithi prudhvigodithi requested a review from a team August 3, 2022 01:04
@prudhvigodithi
Copy link
Copy Markdown
Member Author

Hey @VijayanB can you please review.
Thank you

@prudhvigodithi prudhvigodithi self-assigned this Aug 3, 2022
@prudhvigodithi prudhvigodithi added backport 2.x Backport PR to 2.x branch backport 2.2 backport to 2.2 branch labels Aug 3, 2022
@codecov-commenter
Copy link
Copy Markdown

codecov-commenter commented Aug 3, 2022

Codecov Report

Merging #111 (de60b9a) into main (7995142) will not change coverage.
The diff coverage is n/a.

@@            Coverage Diff            @@
##               main     #111   +/-   ##
=========================================
  Coverage     86.50%   86.50%           
  Complexity      248      248           
=========================================
  Files            37       37           
  Lines           830      830           
  Branches         55       55           
=========================================
  Hits            718      718           
  Misses           87       87           
  Partials         25       25           

Help us with your feedback. Take ten seconds to tell us how you rate us. Have a feature suggestion? Share it here.

Signed-off-by: prudhvigodithi <pgodithi@amazon.com>
@junqiu-lei junqiu-lei merged commit 2e994e0 into opensearch-project:main Aug 3, 2022
opensearch-trigger-bot bot pushed a commit that referenced this pull request Aug 3, 2022
opensearch-trigger-bot bot pushed a commit that referenced this pull request Aug 3, 2022
junqiu-lei pushed a commit that referenced this pull request Aug 3, 2022
Prudhvi Godithi <pgodithi@amazon.com>
(cherry picked from commit 2e994e0)

Co-authored-by: Prudhvi Godithi <pgodithi@amazon.com>
junqiu-lei pushed a commit that referenced this pull request Aug 3, 2022
Prudhvi Godithi <pgodithi@amazon.com>
(cherry picked from commit 2e994e0)

Co-authored-by: Prudhvi Godithi <pgodithi@amazon.com>
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 backport 2.2 backport to 2.2 branch

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants