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

Add NBTExplorer installer #264

Open
wants to merge 1 commit into
base: main
Choose a base branch
from
Open

Conversation

FinnKrestel
Copy link

@FinnKrestel FinnKrestel commented Sep 4, 2023

Type of change

  • New installer
  • Manifest fix
  • Other

Was This Tested Using a Local Repository?

  • Yes
  • No

@orowith2os
Copy link

Please fill out the review properly.

@FinnKrestel
Copy link
Author

Hi, I've used the Epic Games Store Review as a template. Since NBTExplorer works just as well as on native Windows when using recent Mono runtimes, I've given the application the grade "Platinum" for all categories.

Copy link
Member

@mirkobrombin mirkobrombin left a comment

Choose a reason for hiding this comment

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

Looks good to me and the review is ok if there are no information to share

@mirkobrombin
Copy link
Member

Follow-up, just noticed the icon is missing, Can you add one?

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.

3 participants