Open
Conversation
Contributor
Author
|
Looks like the build error is not because of my code but rather the CI itself failing:
|
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
This will help with players stuck in the wilderness. If they are skulled they will lose all their items (or all but 1 item if protecting item), if they are unskulled they will lose all items but 3 (or all but 4 items if protecting item).
This works well from my testing.
I'd also like to add some other commands in the future, only for administrators/developers, such as:
-Adding city names to ::tele command as possible args
-Adding item and NPC names to ::item and ::npc commands
-Adding ::itemsearch and ::npcsearch to show item and NPC IDs
-Adding ::spellbook swap command to quickly swap spellbooks
That's all I can think of off the top of my head but those might be nice to have.