Skip to content

Update repo.json#140

Closed
CelloSerenity wants to merge 1 commit intoStephenDev0:mainfrom
CelloSerenity:patch-3
Closed

Update repo.json#140
CelloSerenity wants to merge 1 commit intoStephenDev0:mainfrom
CelloSerenity:patch-3

Conversation

@CelloSerenity
Copy link
Copy Markdown
Collaborator

@CelloSerenity CelloSerenity commented Apr 17, 2025

Looking amazing, however, it looks like the current update_json.py is adding a second summary of the version underneath app permissions. This PR removes that entry for now, but that should probably get fixed in the python script for the future.

Looking amazing, however, it looks like the current update_json.py is adding a second summary of the version underneath app permissions. This PR removes that entry for now, but that should probably get fixed.
@neoarz
Copy link
Copy Markdown
Contributor

neoarz commented Apr 17, 2025

intersting will look to fix this soon

@CelloSerenity
Copy link
Copy Markdown
Collaborator Author

Looking at the current file, nothing seems amiss that should cause this (I'm only a mediocre programmer so far, so take my words with a grain of salt). Thanks a ton @neoarz !

@neoarz
Copy link
Copy Markdown
Contributor

neoarz commented Apr 17, 2025

appreciate the compliment but i think i see the issue.
the script was based off of the live container repo structure, which is a bit different then the current one we have rn

ill just fix the json file repo structure to look like the one from live containers: https://github.com/LiveContainer/LiveContainer/blob/main/apps.json

@neoarz
Copy link
Copy Markdown
Contributor

neoarz commented Apr 17, 2025

@CelloSerenity could u take a final look at the updated repo format
check to see if all the indentations and stuff are correct
it matches live containers repo format
i didnt leave out any info or anything from the old format

https://github.com/neoarz/StikJIT/blob/main/repo.json

@CelloSerenity
Copy link
Copy Markdown
Collaborator Author

Looks great!

@neoarz
Copy link
Copy Markdown
Contributor

neoarz commented Apr 17, 2025

@CelloSerenity @0-Blu i think we can close this pr as ive updated the format anyways and already added the new release

@CelloSerenity CelloSerenity deleted the patch-3 branch April 17, 2025 21:21
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