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

feat: add Windows standalone archive install option #481

Merged
merged 3 commits into from
Oct 9, 2024
Merged

Conversation

maxrake
Copy link
Contributor

@maxrake maxrake commented Oct 9, 2024

This change adds another Windows standalone "installation" option, in the form of a zip archive that contains all the files needed to run the phylum-ci.exe binary...which is also included in the archive. This option is not as convenient as the single file self-extracting executable since it requires an additional unzipping step. It does have the advantage of not getting flagged by antivirus. This option was added based on direct feedback from an existing customer and directed by @furi0us333.

Testing

The changes from this branch were tested in a Preview workflow run made from a standalone_test branch and can be viewed here. The zip archive did not flag AV and the contents were successfully used on a Windows system to analyze a git repository.

This change adds another Windows standalone "installation" option, in
the form of a zip archive that contains all the files needed to run the
`phylum-ci.exe` binary...which is also included in the archive. This
option is not as convenient as the single file self-extracting
executable since it requires an additional unzipping step. It does have
the advantage of not getting flagged by antivirus. This option was
added based on direct feedback from an existing customer.
@maxrake maxrake self-assigned this Oct 9, 2024
@maxrake maxrake requested a review from a team as a code owner October 9, 2024 15:47
@maxrake maxrake requested a review from cd-work October 9, 2024 15:47
README.md Outdated Show resolved Hide resolved
README.md Outdated Show resolved Hide resolved
README.md Show resolved Hide resolved
@maxrake maxrake requested a review from cd-work October 9, 2024 16:27
README.md Outdated Show resolved Hide resolved
README.md Outdated Show resolved Hide resolved
@maxrake maxrake requested a review from cd-work October 9, 2024 16:30
@maxrake maxrake enabled auto-merge (squash) October 9, 2024 16:32
@maxrake maxrake merged commit 83538a0 into main Oct 9, 2024
13 checks passed
@maxrake maxrake deleted the standalone branch October 9, 2024 16:33
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.

2 participants