Skip to content
This repository has been archived by the owner on May 27, 2023. It is now read-only.

Added MacOS Support with new icon #39

Open
wants to merge 2 commits into
base: dev
Choose a base branch
from
Open

Conversation

p0ryae
Copy link
Contributor

@p0ryae p0ryae commented Apr 6, 2023

discord-tauri throws an error on OSX, because of undefined main function for MacOS based operating systems (ICYMI we handle the main function per operating system). This pr aims to fix that issue. In addition, icon fitting mac aesthetic has been added. I've tested it and it functions just fine in my end.

Bonus: Added serve cli in package.json for running cargo tauri dev through only using yarn serve (or npm run serve)

@@ -1,7 +1,7 @@
{
"name": "discord-tauri",
"scripts": {
"tauri": "tauri"
Copy link
Member

Choose a reason for hiding this comment

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

Thank you for making this, I have one question tough, won't removing "tauri" from the scripts make "yarn tauri" or "npm run tauri" not work?

Copy link
Contributor Author

Choose a reason for hiding this comment

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

I mentioned yarn serve and npm run server in my comment. Would you like me to replace serve with tauri?

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants