Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Jump to bottom

Initialize 2.0.0 Concept. #189

Draft
wants to merge 5 commits into
base: master
Choose a base branch
from
Draft

Initialize 2.0.0 Concept. #189

wants to merge 5 commits into from

Conversation

vLuckyyy
Copy link
Member

No description provided.

@vLuckyyy vLuckyyy marked this pull request as draft September 27, 2024 21:18
vLuckyyy and others added 4 commits September 28, 2024 14:45
@vLuckyyy
GH-187 Use LiteCommands 3x (#191)
381007a 
* GH-187 Use LiteCommands 3x

* GH-187 Separate commands to a different classes

* Update CI to match 2.0 branch.

* Fix.

* Fix command name mistake.

* Follow Mr. Rollczi's Feedback.

* Follow Mr. Rollczi's Feedback v2.

* :trollface:
@vLuckyyy
GH-167 start of normalizing developer api (#195)
9203309 
* GH-167 start of normalizing developer api

* Extract all of services to interfaces

* Add jetbrains annotatiots.

* Fix checkstyle violations

* Follow citralflo feedback.

* Add CancelTagReason

* remove unsed `OTHER` in CancelTagReason.
@CitralFlo @Rollczi
GH-107 Multification setup (#197)
dfc825c 
* Multification setup

* Bump version of multification

* Override adventure API. Bump LiteCommands

* Fix import

* Fix minor issues with notices

* Change BukktiNotice to Notice

* Resolve Rollczi's suggestions

* MAJOR FIX

* uncomment dev.rollczi.litecommands needed for testing

---------

Co-authored-by: Rollczi <[email protected]>
@CitralFlo
GH-200 Add option to exclude Creative mode players from combat (#200)
c8483e5 
* Add creative setting, reformat name of config value

* Extract method

* Devide method to two ifs. Following @Rollczi's advice

* Devide method to two ifs. Following @Rollczi's advice
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers
No reviews
Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
2 participants
@vLuckyyy @CitralFlo