-
Notifications
You must be signed in to change notification settings - Fork 281
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
Adds build script for publishing plugin zip #1921
Adds build script for publishing plugin zip #1921
Conversation
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
You need to modify the script to be executable https://stackoverflow.com/a/40979016/533057
e07b67f
3941d5d
to
e07b67f
Compare
Done. Thank you for pointing this out @peternied ! |
Signed-off-by: Darshit Chanpura <[email protected]>
e07b67f
to
8e7913a
Compare
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Did a little bit of offline debugging, git add --chmod=+x {file}
is the easiest way to go. Just checked the latest version, looks good to me
@DarshitChanpura Please merge the PR when it's ready. |
…1921) (#1923) (cherry picked from commit 192ef35) Co-authored-by: Darshit Chanpura <[email protected]>
…pensearch-project#1921) Signed-off-by: Stephen Crawford <[email protected]>
…pensearch-project#1921) (opensearch-project#1923) (cherry picked from commit 192ef35) Co-authored-by: Darshit Chanpura <[email protected]>
Description
Adds build script for publishing plugin zips to maven repo.
Issues
Check List
- [ ] New functionality includes testing- [ ] New functionality has been documentedBy 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.