Skip to content
This repository was archived by the owner on Feb 28, 2024. It is now read-only.

add VRMLImporter - #93

Open
EIA485 wants to merge 7 commits into
masterfrom
EIA485-patch-14
Open

add VRMLImporter#93
EIA485 wants to merge 7 commits into
masterfrom
EIA485-patch-14

Conversation

@EIA485

@EIA485 EIA485 commented Jul 21, 2022

Copy link
Copy Markdown
Member

mod author is providing a zip as the file, not sure if we should do anything so that down the line an auto installer could install this.

@l-2-j

l-2-j commented Jul 21, 2022

Copy link
Copy Markdown
Member

Missing broken on linux flag I imagine since zip contains .exe files which I guess are dependencies. Haven't checked tho yet. Also I'd probs avoid adding the old version and just the new version so that auditing is less work...?

Comment thread manifest.json
Comment on lines +230 to +236
"artifacts": [
{
"url": "https://github.com/dfgHiatus/NeosVRMLImporter/releases/download/v1.0.1/VRML_Importer_v1.0.1.zip",
"filename": "VRML_Importer_v1.0.1.zip",
"sha256": "727f38e1dfec82ae52867bdbe9b223c33040636adcaf493424860afec4b9c84f"
}
]

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.

Agh, anyone else want to review this? I'm too lazy to review anything that the review script is not set up to automatically download, hash check & decompile x'D

@zkxs

zkxs commented Aug 21, 2022

Copy link
Copy Markdown
Collaborator

No comma after the changelog line.

@zkxs zkxs added the non-mod-author PR Manifest addtion made for somone else's mod label Aug 23, 2022
@l-2-j
l-2-j requested review from XDelta and zkxs September 8, 2022 10:21
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Labels

non-mod-author PR Manifest addtion made for somone else's mod

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants