Skip to content

Conversation

@SidOfThe7Cs
Copy link
Contributor

@SidOfThe7Cs SidOfThe7Cs commented Feb 9, 2026

What

removes the warp delay for non spade guesses, the delay for spade is there to prevent you from warping while the spade particles are still being detected and was not removed, the delay for all guesses was added to prevent you from warping before the arrow particles themselves were detected but this simply doesnt happen as it waits for the chat message before removing the burrow (and triggering a target update allowing warp) which should arrive at the same time as particles and then you have to wait for the warp to actually be registered by the server before you get tped away

Changelog Fixes

  • Fixed warping in diana having an unnecessary delay on non-spade guesses. - SidOfThe7Cs

@github-actions github-actions bot added the Wrong Title/Changelog There is an error in the title or changelog label Feb 9, 2026
@github-actions
Copy link

github-actions bot commented Feb 9, 2026

I have detected some issues with your pull request:

Body issues:
No changes detected in this pull request

Please fix these issues. For the correct format, refer to the pull request template.

@github-actions github-actions bot removed the Wrong Title/Changelog There is an error in the title or changelog label Feb 9, 2026
Copy link
Owner

@hannibal002 hannibal002 left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

What problem does this solve, and are you sure this doesn't cause bugs?

The 400ms warp block was added intentionally to prevent warping immediately after clicking on a burrow. Removing it without explanation makes it hard to evaluate whether this is safe, and what's the reasoning that this won't cause issues?

Also, this is a user-facing behaviour change, so exclude_from_changelog doesn't apply here. From the contributing guidelines

Some changes don't fit any categories. E.g. when reverting pull requests or doing quick fixes to PRs merged immediately beforehand but not yet released in a beta.

This PR needs a changelog entry (likely under Fixes or Improvements) and a filled out What section explaining why removing the cooldown is safe.

@github-actions github-actions bot added the Wrong Title/Changelog There is an error in the title or changelog label Feb 9, 2026
@github-actions
Copy link

github-actions bot commented Feb 9, 2026

I have detected some issues with your pull request:

Body issues:
Change should start with 'Fixed' instead of 'Fix' in text: Fix warping in diana having an unnecessary delay on non-spade guesses.

Please fix these issues. For the correct format, refer to the pull request template.

@github-actions github-actions bot removed the Wrong Title/Changelog There is an error in the title or changelog label Feb 9, 2026
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.

2 participants