Skip to content

Conversation

@CodyIBrealey
Copy link
Contributor

@CodyIBrealey CodyIBrealey commented Nov 21, 2025

added my mod. hope all this works

mp_sanctuary v0.0.1

Did I read the whole README file and understood all of its content?

YES

Did I update verified-mods.json with my new mod's information?

YES

Has a previous version of my mod been previously verified?

NO

Mod description

Literally the third best titanfall 2 map ever created

Thunderstore link

(https://thunderstore.io/c/northstar/p/skrubslayer69/mp_sanctuary/)

added my mod. hope all this works
Copy link
Member

@catornot catornot left a comment

Choose a reason for hiding this comment

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

The mod is to packaged incorrectly,
mods/skrubslayer.mp_sanctuary/mod needs to become mod afaik

@ASillyNeko
Copy link
Contributor

image no mod.json file.

@CodyIBrealey
Copy link
Contributor Author

updated to hopefully fix this. Moved the mod.json file to the correct location, comparing to mp_brick it seems like that particular folder is supposed to be called "mod" rather than "mods", though if it still doesn't work right then that can be corrected.

@catornot
Copy link
Member

catornot commented Nov 22, 2025

Yeah still doesn't work
you need to collapse mods/skrubslayer.mp_sanctuary into the root dir :)
like mp_bricks

fixed the file structure hopefully
@CodyIBrealey
Copy link
Contributor Author

Fixed the file structure as instructed. Hopefully third times the charm.

@catornot
Copy link
Member

yeah ci failed

Copy link
Member

@Alystrasz Alystrasz left a comment

Choose a reason for hiding this comment

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

  • CI verification passes
  • Commit hash is OK
  • Checksum is OK
  • No malicious scripts inside mod itself, and no use of "dangerous" 1 functions

LGTM.

Footnotes

  1. e.g.: Safe I/O stuff and SquirrelHTTP. "dangerous" is a bit too much maybe. Under the assumption that the native implementations are correct and secure, there's nothing dangerous here. However it's always recommended to use more of a Swiss cheese approach and also operate a bit under the assumption that a layer can fail.

@Alystrasz Alystrasz merged commit f5fbc88 into R2Northstar:main Nov 23, 2025
3 checks passed
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.

4 participants