-
Notifications
You must be signed in to change notification settings - Fork 66
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
Add tasks to publish zips for Notifications and Notifications Core plugins #484
Conversation
…ugins Signed-off-by: Mohammad Qureshi <47198598+qreshi@users.noreply.github.com>
I tested locally and running Since we have two backend plugins here, we'll need to make the following changes to the scripts in https://github.com/opensearch-project/opensearch-build: For notifications-core build script:
For notifications build script:
@peterzhuamazon @prudhvigodithi Can you advise that the changes above to the build scripts are in fact what you expect? |
Hey @qreshi yes those 3 lines looks good, also just FYI, here are some PR's to refer |
Hey @qreshi thanks for the PR, please make sure you add |
Signed-off-by: Mohammad Qureshi 47198598+qreshi@users.noreply.github.com
Description
Publish Notifications and Notifications Core zips to maven repo.
Issues Resolved
#474
Check List
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.