pulsar-edit/pulsar
1
1
Fork 0
mirror of https://github.com/pulsar-edit/pulsar.git synced 2024-09-11 11:15:24 +03:00
Code Issues Packages Projects Releases Wiki Activity

Update ADR doc

Browse Source
This commit is contained in:
confused-Techie 2023-08-10 20:36:23 -07:00
parent 68bf168997
commit 87d3986e46
1 changed files with 4 additions and 4 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

8
docs/decisions/005-remove-auto-update-functionality.md
View File

@ -8,14 +8,14 @@ deciders: '@confused-Techie, @Daeraxa, @savetheclocktower, @mauricioszabo'
## Context and Problem Statement
To provide users with some form of autoupdate functionality, many thought we could
adopt the original method used by the Atom team, and defacto method within Electron
adopt the original method used by the Atom team, and de-facto method within Electron
applications, Squirrel. While Squirrel is great for AutoUpdating, it has some big
issues, mainly it requires that every platform has signed binaries, which can be
prohibitively expensive. Additionally, it only supports Windows and macOS.
## Decision Drivers
* Users need a way to fullfill AutoUpdate functionality
* Users need a way to fulfill AutoUpdate functionality
## Considered Options
@ -75,7 +75,7 @@ There was also additional concerns about not deleting the code used for Squirrel
in case we ever did want to return to that behavior, since it already works perfectly, if properly setup.
For that reason, instead of keeping the code within the repo, below will be details about where the code that comprises
of the Squirrel AutoUpdate logic will be kept within Git, so that it can always be retreived if ever needed.
of the Squirrel AutoUpdate logic will be kept within Git, so that it can always be retrieved if ever needed.
* Last Commit Before Removal: `bf60fbe6fc267b737a70d5d39c03cad1629ea128`
* PR Where it was Removed: [`pulsar-edit/pulsar#TBD`]()
* PR Where it was Removed: [`pulsar-edit/pulsar#668`](https://github.com/pulsar-edit/pulsar/pull/668)