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

Added MacOS Support with new icon#39

Open
p0ryae wants to merge 2 commits into
DiscordTauri:devfrom
p0ryae:dev
Open

Added MacOS Support with new icon#39
p0ryae wants to merge 2 commits into
DiscordTauri:devfrom
p0ryae:dev

Conversation

@p0ryae
Copy link
Copy Markdown
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)

Comment thread package.json
{
"name": "discord-tauri",
"scripts": {
"tauri": "tauri"
Copy link
Copy Markdown
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
Copy Markdown
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