Skip to content

test(oxfmt): remove unnecessary timeout from Tailwind CSS test#19050

Merged
graphite-app[bot] merged 1 commit intomainfrom
remove-tailwind-test-timeout
Feb 6, 2026
Merged

test(oxfmt): remove unnecessary timeout from Tailwind CSS test#19050
graphite-app[bot] merged 1 commit intomainfrom
remove-tailwind-test-timeout

Conversation

@Boshen
Copy link
Member

@Boshen Boshen commented Feb 6, 2026

Summary

🤖 Generated with Claude Code

Copilot AI review requested due to automatic review settings February 6, 2026 10:28
@github-actions github-actions bot added A-cli Area - CLI A-formatter Area - Formatter C-bug Category - Bug labels Feb 6, 2026
Copy link
Contributor

Copilot AI left a comment

Choose a reason for hiding this comment

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

Pull request overview

This PR reverts the timeout workaround that was added in PR #18339 to address Windows CI test failures. The change removes the 30-second timeout and explanatory comment from the first Tailwind CSS sorting test, reverting it back to the default 10-second timeout.

Changes:

  • Remove 30-second timeout from the first Tailwind CSS test that triggers plugin initialization
  • Remove the explanatory comment about Windows CI performance issues

💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.

@Boshen Boshen added the 0-merge Merge with Graphite Merge Queue label Feb 6, 2026
@Boshen Boshen changed the title fix(oxfmt): remove unnecessary timeout from Tailwind CSS test test(oxfmt): remove unnecessary timeout from Tailwind CSS test Feb 6, 2026
@github-actions github-actions bot added the C-test Category - Testing. Code is missing test cases, or a PR is adding them label Feb 6, 2026
Copy link
Member Author

Boshen commented Feb 6, 2026

Merge activity

## Summary
- Remove the 30s timeout and associated comment from the first Tailwind CSS sorting test, added in #18339

🤖 Generated with [Claude Code](https://claude.com/claude-code)
@graphite-app graphite-app bot force-pushed the remove-tailwind-test-timeout branch from 8740402 to 9da6b0b Compare February 6, 2026 10:56
@graphite-app graphite-app bot merged commit 9da6b0b into main Feb 6, 2026
19 checks passed
@graphite-app graphite-app bot deleted the remove-tailwind-test-timeout branch February 6, 2026 11:02
@graphite-app graphite-app bot removed the 0-merge Merge with Graphite Merge Queue label Feb 6, 2026
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

A-cli Area - CLI A-formatter Area - Formatter C-bug Category - Bug C-test Category - Testing. Code is missing test cases, or a PR is adding them

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants