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

Update gemspec to include signing key path #45

Merged
merged 1 commit into from
Dec 6, 2021

Conversation

VijayanB
Copy link
Member

@VijayanB VijayanB commented Dec 3, 2021

Signed-off-by: Vijayan Balasubramanian balasvij@amazon.com

Description

By default, signing key is expected to be at
root of project with name gem-private_key.pem.

Include only if command is gem and argument is gemspec and signing key exists.

Issues Resolved

#23

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.

By default, signing key is expected to be at
root of project with name gem-private_key.pem.

Include only if command is gem and argument is gemspec and signing key exists.

Signed-off-by: Vijayan Balasubramanian <balasvij@amazon.com>
Copy link
Contributor

@jayeshathila jayeshathila left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Why empty release/build.sh ? Any usage for this ?

@VijayanB
Copy link
Member Author

VijayanB commented Dec 6, 2021

Why empty release/build.sh ? Any usage for this ?

It is not empty, I changed the permission so that it is executable. This will be used by infra to build all gems

@jayeshathila
Copy link
Contributor

Why empty release/build.sh ? Any usage for this ?

It is not empty, I changed the permission so that it is executable. This will be used by infra to build all gems

Ah. As the git only tracks file's executable nature hence not showing any difference. Got it.

Copy link
Member

@vamshin vamshin left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM! Thanks

@VijayanB VijayanB merged commit d53fac3 into opensearch-project:main Dec 6, 2021
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants