Skip to content

Conversation

@Loup-Garou911XD
Copy link
Member

No description provided.

@Loup-Garou911XD Loup-Garou911XD changed the title Pre release 2 add black formatting Aug 16, 2025
@Loup-Garou911XD
Copy link
Member Author

Loup-Garou911XD commented Aug 16, 2025

@brostosjoined one issue
we need to make a new release for every plugin that is getting formatted which is 116 or so files
checkout https://github.com/Loup-Garou911XD/plugin-manager

@Loup-Garou911XD
Copy link
Member Author

maybe we should get the autometa branch merged first so we can maybe update the checksum automatically somehow???

@Loup-Garou911XD
Copy link
Member Author

idk how this will work out

@brostosjoined
Copy link
Member

hmmm what if we just ignore the changes made and comment out the Apply Version Metadata workflow then after the changes we revert back

@brostosjoined
Copy link
Member

or i can write a simple script to update them

@Loup-Garou911XD
Copy link
Member Author

hmmm what if we just ignore the changes made and comment out the Apply Version Metadata workflow then after the changes we revert back

eh i dont understand how that will work out
the checksum must match or there will be problems

@brostosjoined
Copy link
Member

run the script and it will update the version of all the plugins

@Loup-Garou911XD
Copy link
Member Author

@rikkolovescats should we go ahead with this or is this a really bad idea?

@vishal332008
Copy link
Member

indent=2? ugh.
And let mod devs have some freedom on how they make the things they make, just make rules like it should be understandable enough and not contain anything bad

@Loup-Garou911XD
Copy link
Member Author

checking this out after 4 months
i think the best way is to just switch to black instead of autopep8
and when the author decides to release a new version of their plugin, it will be formatted with black
otherwise it stays as is

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