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

fix tag:with of toolshed command #34900

Open
wants to merge 3 commits into
base: master
Choose a base branch
from
Open

fix tag:with of toolshed command #34900

wants to merge 3 commits into from
+2 −5

Conversation

KillanGenifer
Copy link

About the PR

Deleted CommandInvocationContext from the command that causes stack overflow and server crash. no code - no crash

Requirements

@github-actions github-actions bot added S: Needs Review Status: Requires additional reviews before being fully accepted S: Untriaged Status: Indicates an item has not been triaged and doesn't have appropriate labels. size/XS Denotes a PR that changes 0-9 lines. labels Feb 5, 2025
@ArtisticRoomba ArtisticRoomba added A: Core Tech Area: Underlying core tech for the game and the Github repository. T: Bugfix Type: Bugs and/or bugfixes P2: Raised Priority: Item has a raised priority, indicating it might get increased maintainer attention. D3: Low Difficulty: Some codebase knowledge required. A: Toolshed Area: Toolshed, the command interpreter, related changed or fixes and removed S: Untriaged Status: Indicates an item has not been triaged and doesn't have appropriate labels. A: Core Tech Area: Underlying core tech for the game and the Github repository. labels Feb 5, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@DrSmugleaf DrSmugleaf Awaiting requested review from DrSmugleaf DrSmugleaf is a code owner

@nikthechampiongr nikthechampiongr Awaiting requested review from nikthechampiongr nikthechampiongr is a code owner

@crazybrain23 crazybrain23 Awaiting requested review from crazybrain23 crazybrain23 is a code owner

Assignees
No one assigned
Labels
A: Toolshed Area: Toolshed, the command interpreter, related changed or fixes D3: Low Difficulty: Some codebase knowledge required. P2: Raised Priority: Item has a raised priority, indicating it might get increased maintainer attention. S: Needs Review Status: Requires additional reviews before being fully accepted size/XS Denotes a PR that changes 0-9 lines. T: Bugfix Type: Bugs and/or bugfixes
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
2 participants
@KillanGenifer @ArtisticRoomba