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

build: add package Github Action #1888

Merged
merged 2 commits into from
Sep 23, 2021
Merged

Conversation

hicom150
Copy link
Contributor

@hicom150 hicom150 commented Nov 6, 2020

What is the purpose of this pull request? (put an "X" next to item)

[ ] Documentation update
[ ] Bug fix
[x] New feature
[ ] Other, please explain:

What changes did you make? (Give an overview)
Add package Github Action. With this action that can be manually triggered via Actions menu, Linux, MacOS and Windows build will be launched and linux.zip, macos.zip and windows.zip artifacts will be generated.

It is important to note that MacOS and Windows installers are currently not signed, so we will have to add this in next steps 😅. I see this PR as the first step towards package build automation 😉

Which issue (if any) does this pull request address?
#1849

Is there anything you'd like reviewers to focus on?

Copy link
Member

@feross feross left a comment

Choose a reason for hiding this comment

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

LGTM! One possible improvement – is there any way to remove some of the duplication across jobs?

@DiegoRBaquero
Copy link
Member

I believe the cache could be done after npm install, so it's just running the package command

@dsernst
Copy link
Contributor

dsernst commented Jun 26, 2021

hey @hicom150 — this is super cool.

I'd love to help review but can you help me understand how to best do that to ensure this is working correctly?

I'm not as familiar with Github Actions (yet). Is it possible to test these builds before merging?
Where do the Github Actions artifacts go, exactly?

@github-actions
Copy link

Is this still relevant? If so, what is blocking it? Is there anything you can do to help move it forward?

@dsernst
Copy link
Contributor

dsernst commented Aug 25, 2021

@github-actions Yes, still relevant.

This would be very cool & could significantly unblock / speed up development.

@DiegoRBaquero DiegoRBaquero changed the title Add package Github Action build: add package Github Action Sep 23, 2021
@DiegoRBaquero DiegoRBaquero merged commit c764744 into master Sep 23, 2021
@DiegoRBaquero DiegoRBaquero deleted the add_package_github_action branch September 23, 2021 00:52
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

5 participants