Skip to content

Conversation

@Zazk0u
Copy link
Contributor

@Zazk0u Zazk0u commented Jul 14, 2025

No description provided.

@Zazk0u
Copy link
Contributor Author

Zazk0u commented Jul 14, 2025

For context i made a python mod, i hope i did not messed this up

@apple1417
Copy link
Contributor

apple1417 commented Jul 14, 2025

Few things about your pyproject:

  • All the base stuff under project is standard python packaging fields. project.name may not contain spaces. That's why you can overwrite it under the sdkmod section. This is the one thing that actually needs changing, rest are just suggestions.
  • The SDK assumes the first author is the main author, you may want to put yourself first
  • Your pyproject doesn't specify the license, your repo uses gpl3

And then suggestions for your code:

  • You probably meant to set the option descriptions, not description titles
  • You only need to use the global statement if you're setting a global, you can always read one
  • In request_skill_upgrade it sounds like you'd be better off using unrealsdk.find_enum than magic numbers
  • In fake_level_five, that getattr is useless, it's just setting func back to what it already was.
  • You might consider splitting fake_level_five into a seperate pre/post hook, so that you don't need to block execution. This will play better if any other mod hooks the same functions.

@Zazk0u
Copy link
Contributor Author

Zazk0u commented Jul 14, 2025 via email

@Zazk0u
Copy link
Contributor Author

Zazk0u commented Jul 15, 2025

I've updated my pyproject and followed your suggestions.
Hopefully i did not mess anything up with the merge branch, i'm not super into github.

@apple1417 apple1417 merged commit feaa24d into bl-sdk:master Jul 15, 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.

2 participants