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 Unity Explorer #51

Merged
merged 3 commits into from
Dec 28, 2023
Merged

Add Unity Explorer #51

merged 3 commits into from
Dec 28, 2023

Conversation

arthomnix
Copy link
Contributor

Unity Explorer is a tool for exploring and modifying the internals of Unity games (including KSP2) on the fly, useful for mod developers or just for poking around in the game's internals.

@HebaruSan
Copy link
Member

Hi @Falki-git, there's some interest in having Unity Explorer in CKAN, and we've heard that your fork is the preferred one among KSP2 modders. Would it be OK with you if we added it? Adding the original fork has issues with trying to get informed consent from someone who doesn't care about KSP2 or CKAN.

Side note that the swinfo.json has a broken version_check link, so we'd have to take the newly added $vref back out for this to work.

image

@cheese3660
Copy link
Contributor

It seems as if @Falki-git does not put the remote swinfo in the repository? (Referencing the netkan error)

@jan-bures
Copy link
Contributor

Yep, I just submitted a PR for that: KSP2Community/UnityExplorer#1

@cheese3660
Copy link
Contributor

That should be everything, shall I go ahead and merge this?

@Falki-git
Copy link
Contributor

@HebaruSan, of course, we can add that fork of Unity Explorer to CKAN.
Additionally, we've agreed to transfer ownership to KSP2Community org, so the new link to the source code is:
https://github.com/KSP2Community/UnityExplorer

What else is needed to make this happen?

@Falki-git
Copy link
Contributor

That should be everything, shall I go ahead and merge this?

Please do.

@cheese3660
Copy link
Contributor

Got it, and I already edited the netkan to point to the org

Copy link
Member

@HebaruSan HebaruSan left a comment

Choose a reason for hiding this comment

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

@cheese3660 Sure, go ahead. 👍

@cheese3660 cheese3660 merged commit 31e0026 into KSP-CKAN:main Dec 28, 2023
1 check passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

5 participants