chocolatey-community / chocolatey-packages

Chocolatey Community Maintainers Team Packages - packages that are managed and maintained by core community team for community package repository (https://community.chocolatey.org/packages)
https://community.chocolatey.org/profiles/chocolatey-community
Apache License 2.0
447 stars 377 forks source link

(1Password8) Adding 1Password 8 package to chocolatey community including automatic update #2462

Closed chrrobi closed 4 months ago

chrrobi commented 5 months ago

Description

Created a 1Password 8 package that can be installed through chocolatey. It is set at 8.10.27. This includes a relatively standard install, a custom uninstall (as the automatic one fails as it looks for 1Password in the wrong place), and provide an update which will repackage for new versions.

This package downloads the MSI from the 1password site and does not package the binary.

NOTE 1password removes older version eventually so some of these will become dead as they move forward in time.

Motivation and Context

1Password made a significant change in update approach when they moved to one password 8. This package attempts to take those into account to add the ability to install the newest version of 1password (currently only 1password 7 is supported and has not updated in a long time).

There was an open issue: https://github.com/chocolatey-community/chocolatey-packages/issues/1773 A PR that was closed: https://github.com/chocolatey-community/chocolatey-packages/pull/2061 Another closed PR: https://github.com/chocolatey-community/chocolatey-packages/pull/1972

How Has this Been Tested?

I have packed this locally and then: -installed, and uninstalled, -installed, updated, upgraded, uninstalled -Had pushed it under a different name as a package where it passed all automated tests

Both use cases worked correctly.

The update.ps1 will

Checklist:

AppVeyorBot commented 5 months ago

:white_check_mark: Package verification completed without issues. PR is now pending human review

chrrobi commented 5 months ago

Hello, just following up, is there anything specific you need looked at or reworked for this pull? I am hoping to be able to deploy it soonish.

Thanks for your help and reviews.

Best Regards

AppVeyorBot commented 4 months ago

:x: Package verification failed, please review the Appveyor Logs and the provided Artifacts before requesting a human reviewer to take a look.