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

Correct desktop file category #39

Merged
merged 1 commit into from
Dec 13, 2022
Merged

Correct desktop file category #39

merged 1 commit into from
Dec 13, 2022

Conversation

yochananmarqos
Copy link
Contributor

@yochananmarqos yochananmarqos commented Dec 13, 2022

❯ desktop-file-validate com.vysp3r.ProtonPlus.desktop
com.vysp3r.ProtonPlus.desktop: error: value "Games;" for key "Categories" in group "Desktop Entry" contains an unregistered value "Games"; values extending the format should start with "X-"
com.vysp3r.ProtonPlus.desktop: hint: value "Games;" for key "Categories" in group "Desktop Entry" does not contain a registered main category; application might only show up in a "catch-all" section of the application menu

EDIT: Sorry, missed the template. The above is self-explanatory.

```
❯ desktop-file-validate com.vysp3r.ProtonPlus.desktop
com.vysp3r.ProtonPlus.desktop: error: value "Games;" for key "Categories" in group "Desktop Entry" contains an unregistered value "Games"; values extending the format should start with "X-"
com.vysp3r.ProtonPlus.desktop: hint: value "Games;" for key "Categories" in group "Desktop Entry" does not contain a registered main category; application might only show up in a "catch-all" section of the application menu
```

Signed-off-by: Mark Wagie <[email protected]>
@Vysp3r
Copy link
Owner

Vysp3r commented Dec 13, 2022

When I ran that command I didn't have any error ouputs, that's weird. That's a good thing to fix tho, thank you!

@yochananmarqos
Copy link
Contributor Author

yochananmarqos commented Dec 13, 2022

I just have another pending issue which I want to fix first before the next update.

...and while we're at it: #39 😉

Make sense?

@Vysp3r Vysp3r merged commit cc70217 into Vysp3r:main Dec 13, 2022
@yochananmarqos yochananmarqos deleted the patch-1 branch December 13, 2022 23:46
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.

2 participants