Skip to content

Add fabric support#31

Merged
lucko merged 6 commits into
LuckPerms:masterfrom
PotatoPresident:fabric-support
Feb 21, 2022
Merged

Add fabric support#31
lucko merged 6 commits into
LuckPerms:masterfrom
PotatoPresident:fabric-support

Conversation

@PotatoPresident
Copy link
Copy Markdown
Contributor

Added a Fabric sub-project for the Fabric Placeholder API
Switched from build script from Maven to Gradle

"chat": "<%luckperms:prefix% ${player} %luckperms:suffix% - %luckperms:context% - %luckperms:highest_group_by_weight% [%luckperms:groups%] - In People: %luckperms:in_group/people% - In aaa: %luckperms:in_group/aaa%> ${message}"

image

@eliskvitka
Copy link
Copy Markdown

👍

@lucko
Copy link
Copy Markdown
Member

lucko commented Feb 16, 2022

Hi! thanks for this :)

Would you be able to add license headers to the new classes? (just copy and paste from one of the existing classes & make sure there's an empty line beneath)

Happy to merge this once that's done

@PotatoPresident
Copy link
Copy Markdown
Contributor Author

PotatoPresident commented Feb 18, 2022

Hi! thanks for this :)

Would you be able to add license headers to the new classes? (just copy and paste from one of the existing classes & make sure there's an empty line beneath)

Happy to merge this once that's done

Done
@lucko

@lucko lucko merged commit d1f7ebb into LuckPerms:master Feb 21, 2022
@lucko
Copy link
Copy Markdown
Member

lucko commented Feb 21, 2022

Thanks for this :)

I've also updated the website so it'll show up there - LuckPerms/LuckPermsWeb@c52f272

@PotatoPresident PotatoPresident deleted the fabric-support branch February 24, 2022 16:03
@PotatoPresident
Copy link
Copy Markdown
Contributor Author

Thanks for this :)

I've also updated the website so it'll show up there - LuckPerms/LuckPermsWeb@c52f272

Thanks, just opened a PR for the wiki as well.

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