* include check-run in workflows where secrets are used
* revert changes in build-cli workflow and add check-run to codecov
* assert token permissions
* include required permissions
* re-arrange permissions in alphabetical order
---------
Co-authored-by: Matt Bishop <mbishop@bitwarden.com>
* include check-run in workflows where secrets are used
* revert changes in build-cli workflow and add check-run to codecov
* assert token permissions
---------
Co-authored-by: Matt Bishop <mbishop@bitwarden.com>
Utilizing resources based on thread availability improves performance vs fixed threading.
The linked issue for CI crashes has not been closed. Once this is solves, `runInBand` may improves performance in CI.
* Get node version from .nvmrc and use in setup-node action
* Fix substitution
* Fix substitution 2
* Fix substitution 3
* Add ls
* Add ls
* Override working directory
* Fix override working directory
* Fix
* Add ls
* Checkout repo
Upgrade Electron to version 24, node to 18 and npm to 9. Electron changed to using node 18 in 23, with node 18 using npm 9 as default.
There doesn't seem to be any breaking changes except the deprecation of Windows 7, 8.1, and Server 2012. A somewhat undocumented breaking change was that elements in the title bar are now draggable which broke the account switching selector. Resolved by adding a no-drag css rule.
- electronjs.org/blog/electron-22-0
- electronjs.org/blog/electron-23-0
- electronjs.org/blog/electron-24-0
* Add tests step to build pipelines
* RUn tests for safari
* Remove tests step to build pipelines
* Run tests on root dir on pr
* Revert "Remove tests step to build pipelines"
This reverts commit 497bd86620.
* CHange name
* Remove working directory
* Add manual trigger
* Comment out cache
* Try to fix
* Revert "Try to fix"
This reverts commit 93d404b7d5.
* Disable failing test
