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 package #1240

Merged
merged 2 commits into from
Nov 5, 2019
Merged

Fix electron package #1240

merged 2 commits into from
Nov 5, 2019

Conversation

campersau
Copy link
Collaborator

use latest electron packager and build them in CI.

fixes #593 for the second time :)

- git config --global user.email "test@testy.com"
- git config --global user.name "Test testy"
- git version
- npm run build
after_success:
- DISPLAY= npm run package
Copy link
Collaborator Author

Choose a reason for hiding this comment

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

This took me 1 1/2 days to find out 🤦‍♂

@campersau campersau mentioned this pull request Nov 3, 2019
@campersau campersau merged commit 6bf56cb into master Nov 5, 2019
@campersau campersau deleted the electron-package branch November 5, 2019 12:55
This was referenced Nov 5, 2019
@aitboudad
Copy link

@campersau I'm unable to use copy and paste when using electron app, is it intended?

@campersau
Copy link
Collaborator Author

@aitboudad can you please open a new issue?
Also please include the ungit version and os because for me copy & paste from and to ungit is working with ungit electron 1.5.0 on windows 10.

@aitboudad
Copy link

done #1251

@campersau campersau mentioned this pull request May 3, 2020
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Package Ungit as an Electron App
3 participants