Skip to content

Conversation

@bmuenzenmeyer
Copy link
Contributor

Description

Validation

Related Issues

fast-follow to #8175

Check List

  • I have read the Contributing Guidelines and made commit messages that follow the guideline.
  • I have run pnpm format to ensure the code follows the style guide.
  • I have run pnpm test to check if all tests are passing.
  • I have run pnpm build to check if the website builds without errors.
  • I've covered new added functionality with unit tests if necessary.

Copilot AI review requested due to automatic review settings November 7, 2025 18:09
@bmuenzenmeyer bmuenzenmeyer requested a review from a team as a code owner November 7, 2025 18:09
@vercel
Copy link

vercel bot commented Nov 7, 2025

The latest updates on your projects. Learn more about Vercel for GitHub.

Project Deployment Preview Updated (UTC)
nodejs-org Ready Ready Preview Nov 7, 2025 6:11pm

@bmuenzenmeyer bmuenzenmeyer changed the title resolve prettier issues resolve prettier issues on main Nov 7, 2025
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 applies automated Tailwind CSS class ordering to CSS module files using the prettier-plugin-tailwindcss plugin. The changes standardize the order of utility classes within @apply directives across the codebase to follow Tailwind's recommended class ordering conventions.

  • Reorders Tailwind utility classes within @apply directives
  • Standardizes class ordering across all CSS module files
  • No functional changes to styling or behavior

Reviewed Changes

Copilot reviewed 12 out of 12 changed files in this pull request and generated no comments.

Show a summary per file
File Description
packages/ui-components/src/Common/Select/index.module.css Reorders utility classes in .trigger, .item, .text, and .dropdown selectors
packages/ui-components/src/Common/BaseLinkTabs/index.module.css Swaps order of mt-10 and mb-6 in .tabsList
apps/site/components/Common/Searchbox/index.module.css Reorders positioning classes in .modalWrapper and .modalInner
apps/site/components/Common/Searchbox/SlidingChatPanel/index.module.css Swaps top-2 and right-6 in .slidingPanelCloseButton
apps/site/components/Common/Searchbox/SearchResults/index.module.css Swaps mt-4 and mb-3 in .searchResultsGroupTitle
apps/site/components/Common/Searchbox/SearchItem/index.module.css Moves outline-none after duration-300
apps/site/components/Common/Searchbox/Search/index.module.css Reorders positioning and spacing classes in svg and input selectors
apps/site/components/Common/Searchbox/MobileTopBar/index.module.css Reorders classes in .topBarArrow, .topBarTabs, and .topBarTabActive
apps/site/components/Common/Searchbox/EmptyResults/index.module.css Reorders classes in multiple selectors including .emptyResultsTitle and .noResultsWrapper
apps/site/components/Common/Searchbox/DocumentLink/index.module.css Reorders text truncation and overflow classes in .documentTitle
apps/site/components/Common/Searchbox/ChatSources/index.module.css Reorders text truncation and overflow classes in .chatSourceTitle
apps/site/components/Common/Searchbox/ChatInput/index.module.css Reorders padding classes and moves whitespace-nowrap in utility classes

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

@avivkeller avivkeller enabled auto-merge November 7, 2025 18:12
@codecov
Copy link

codecov bot commented Nov 7, 2025

❌ 1 Tests Failed:

Tests completed Failed Passed Skipped
53 1 52 0
View the full list of 1 ❄️ flaky test(s)
test::custom references

Flake rate in main: 100.00% (Passed 0 times, Failed 1 times)

Stack Traces | 0.00607s run time
[Error [ERR_TEST_FAILURE]: Expected values to be strictly deep-equal:
+ actual - expected

+ [
+   'Invalid type reference: {custom}'
+ ]
- []
] {
  code: 'ERR_TEST_FAILURE',
  failureType: 'testCodeFailure',
  cause: AssertionError [ERR_ASSERTION]: Expected values to be strictly deep-equal:
  + actual - expected
  
  + [
  +   'Invalid type reference: {custom}'
  + ]
  - []
  
      at testRule (file:.../runner/work/nodejs.org/nodejs..../rules/__tests__/utils.mjs:31:10)
      at TestContext.<anonymous> (file:.../runner/work/nodejs.org/nodejs..../rules/__tests__/invalid-type-reference.test.mjs:49:7)
      at Test.runInAsyncScope (node:async_hooks:211:14)
      at Test.run (node:internal/test_runner/test:979:25)
      at Suite.processPendingSubtests (node:internal/test_runner/test:677:18)
      at Test.postRun (node:internal/test_runner/test:1090:19)
      at Test.run (node:internal/test_runner/test:1018:12)
      at async Suite.processPendingSubtests (node:internal/test_runner/test:677:7) {
    generatedMessage: true,
    code: 'ERR_ASSERTION',
    actual: [ 'Invalid type reference: {custom}' ],
    expected: [],
    operator: 'deepStrictEqual'
  }
}

To view more test analytics, go to the Test Analytics Dashboard
📋 Got 3 mins? Take this short survey to help us improve Test Analytics.

@avivkeller
Copy link
Member

@bmuenzenmeyer feel free to merge manually. I'm working on pinning doc-kit now

@bmuenzenmeyer
Copy link
Contributor Author

merging this

@bmuenzenmeyer bmuenzenmeyer merged commit 3118af2 into main Nov 7, 2025
17 of 18 checks passed
@bmuenzenmeyer bmuenzenmeyer deleted the main-fast-follow-prettier branch November 7, 2025 18:28
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.

4 participants