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

Adds Windows Installer (MSI) to the release #332

Merged
merged 5 commits into from
Aug 6, 2021
Merged

Conversation

codefromthecrypt
Copy link
Contributor

This adds func-e.msi manually until goreleaser/goreleaser#1295

This also adds basic testing of the msi as a part of e2e.

@codefromthecrypt
Copy link
Contributor Author

My approach to testing this will be to create a branch off this, remove the homebrew update, and do a dummy release of 0.6.1. If that works, I will delete the dummy release and we are all good!

This adds func-e.msi manually until goreleaser/goreleaser#1295

This also adds basic testing of the msi as a part of e2e.

Signed-off-by: Adrian Cole <[email protected]>
Signed-off-by: Adrian Cole <[email protected]>
@codefromthecrypt
Copy link
Contributor Author

Signed-off-by: Adrian Cole <[email protected]>
@codefromthecrypt
Copy link
Contributor Author

Signed-off-by: Adrian Cole <[email protected]>
@codefromthecrypt
Copy link
Contributor Author

That worked except a glitch as when verifying the install manually in windows, I noticed the control panel mentioned an incorrect version of func-e. fixed

3rd time's a charm https://github.com/tetratelabs/func-e/actions/runs/1103690668

Signed-off-by: Adrian Cole <[email protected]>
@codefromthecrypt
Copy link
Contributor Author

4th time was the charm https://github.com/tetratelabs/func-e/actions/runs/1103702795

image

known issue is that until #320 a blue warning and windows defender notices will happen on install
image

@codefromthecrypt codefromthecrypt merged commit c5f1d3c into master Aug 6, 2021
@codefromthecrypt codefromthecrypt deleted the msi-release branch August 6, 2021 03:15
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.

1 participant