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

GitHub actions - Build Release Binaries #46

Closed
wants to merge 5 commits into from
Closed

Conversation

elvzan
Copy link

@elvzan elvzan commented Mar 26, 2024

Description of change

Created a git hub action event based on 'release' to build binaries and upload it to the Release Artifacts

Notes

Upon creation/publication of a Release, the build release binary will create and upload a Windows, Mac, and Ubuntu binary.

Issue

Resolves #40

@jamolina
Copy link
Contributor

Thank you for your contribution @elvzan. Our team will take a look at this and get back to you with any comments.

@elvzan
Copy link
Author

elvzan commented Mar 28, 2024

@jamolina Let me know if you have any questions

@jamolina
Copy link
Contributor

jamolina commented Apr 5, 2024

@elvzan thank you! Sorry for the delay. We will get back to you soon

@elvzan
Copy link
Author

elvzan commented Apr 10, 2024

@jamolina No worries. Any eta on approving the pull request?

@jamolina
Copy link
Contributor

Hey @elvzan, thank you again for your contribution, and apologies for the delay. Your code looks great and covers all 3 supported platforms (Linux, MacOs, and Windows). At AWS we are constantly raising the bar on security. As part of this commitment, we are required to sign our binaries so our customers are sure when a tool is coming from AWS. Unfortunately, because of this, we cannot accept your contribution at this time. We hope this does not discourages you from any future contributions on this or other projects. If you’re interested in contributing in another feature of the tool, feel free to create an issue or comment on an existing one about your plans so we can discuss it further.

@jamolina jamolina closed this Apr 15, 2024
@elvzan
Copy link
Author

elvzan commented Apr 15, 2024

@jamolina thanks for getting back on this. Will you aws be looking to extend the pull request to sign the binaries?

@jamolina
Copy link
Contributor

@elvzan unfortunately, our signing process is a little different, so we won't be able to re-use this PR

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.

Add Binary
2 participants