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

upgrades to an intermediate version of electron that fixes security #405

Merged
merged 1 commit into from
Apr 16, 2024

Conversation

nhayfield
Copy link
Collaborator

@nhayfield nhayfield commented Apr 16, 2024

Summary

Based on the Dependabot pull request but added some additional updates
Tested Start, Build and Package working on Ubuntu.
Someone should probably test that yarn package is working on macs.

Note this does not update Electron to the latest version which is 30.0. That does break everything.

Related issues

Checklist

  • reference any related issues
  • updated docs
  • updated unit tests
  • updated UPGRADING.md
  • add appropriate tag (improvement / bug / etc)
  • ready for review

@nhayfield nhayfield requested review from a team as code owners April 16, 2024 14:09
@nhayfield nhayfield requested review from calebdoxsey and removed request for a team April 16, 2024 14:09
@nhayfield nhayfield added the dependencies Pull requests that update a dependency file label Apr 16, 2024
@nhayfield nhayfield enabled auto-merge (squash) April 16, 2024 14:09
@nhayfield nhayfield requested review from kenjenkins and removed request for a team and calebdoxsey April 16, 2024 14:10
Copy link
Contributor

@kenjenkins kenjenkins left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Thanks, yarn package seems to be working for me on macOS too.

@nhayfield nhayfield merged commit 904a970 into main Apr 16, 2024
5 checks passed
@nhayfield nhayfield deleted the intermediateElectronForSecurity branch April 16, 2024 17:33
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
dependencies Pull requests that update a dependency file
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

2 participants