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

Bundle build cleanup and code improvements #150

Merged
merged 2 commits into from
Aug 6, 2021

Conversation

dblock
Copy link
Member

@dblock dblock commented Aug 6, 2021

Description

  • Added SPDX headers.
  • Added missing set -e.
  • Use @dataclass to make the python more better closer to the beautiful Ruby :)

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.

@dblock dblock changed the title Bundle build cleanup Bundle build cleanup and code improvements Aug 6, 2021
dblock added 2 commits August 6, 2021 16:35
Signed-off-by: dblock <dblock@amazon.com>
Signed-off-by: dblock <dblock@amazon.com>
@dblock dblock force-pushed the bundle-build-cleanup branch from 33f70d0 to 1c594c9 Compare August 6, 2021 16:35
@dblock dblock requested review from mch2 and peternied August 6, 2021 16:36
@dblock dblock mentioned this pull request Aug 6, 2021
1 task
@dblock dblock mentioned this pull request Aug 6, 2021
1 task
@dblock dblock merged commit ab50a96 into opensearch-project:main Aug 6, 2021
@dblock dblock deleted the bundle-build-cleanup branch August 11, 2021 18:55
alborotogarcia pushed a commit to alborotogarcia/opensearch-build that referenced this pull request Sep 7, 2021
* Added missing set -e and SPDX headers.
* Use dataclass.

Signed-off-by: dblock <dblock@amazon.com>
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