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

Fix electron builds in CI #211

Closed

Conversation

TomAFrench
Copy link
Contributor

Builds of the desktop app are currently failing in CI but work locally. It seems like this has something to do with migrating to yarnv3

@TomAFrench TomAFrench force-pushed the fix-electron-builds branch from 7555f96 to c0837ec Compare August 25, 2022 15:45
@TomAFrench TomAFrench added the do not merge Don't merge this!!! label Aug 25, 2022
@TomAFrench TomAFrench force-pushed the fix-electron-builds branch 2 times, most recently from 225767d to 9bbd54f Compare August 31, 2022 23:07
@netlify
Copy link

netlify bot commented Dec 29, 2022

Deploy Preview for actualbudget canceled.

Name Link
🔨 Latest commit bac1f65
🔍 Latest deploy log https://app.netlify.com/sites/actualbudget/deploys/63c2e8bca95a820008ca1b4c

@MatissJanis
Copy link
Member

Seems that this is not yet ready to be merged. CI jobs are failing.

I'll leave it open for now in case anyone wants to pick it up.

@MatissJanis
Copy link
Member

👋 Thanks for starting this initiative. It's quite a complex one.

Given this has been stale for quite some time now - I'll close it. But please feel free to reopen if you resume working on it.

Also created a tracking ticket so we could use this work as a point of reference in case anyone else wants to pick it up.

#468

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
do not merge Don't merge this!!!
Projects
Status: Fixed
Development

Successfully merging this pull request may close these issues.

3 participants