Skip to content

Upgrade Nextjs and remove eslint and prettier#116

Open
gauravwarale wants to merge 2 commits into
mainfrom
CU-86e24dg77_Heimdall-Nextjs-version-upgrade-and-add-oxclint-and-formatter
Open

Upgrade Nextjs and remove eslint and prettier#116
gauravwarale wants to merge 2 commits into
mainfrom
CU-86e24dg77_Heimdall-Nextjs-version-upgrade-and-add-oxclint-and-formatter

Conversation

@gauravwarale

Copy link
Copy Markdown
Contributor

This pull request migrates the project from ESLint and Prettier to the new Oxlint and Oxfmt tools for linting and formatting, and updates related configuration and scripts. It also includes dependency updates and several minor code and style cleanups across components.

Tooling migration and configuration:

  • Replaced ESLint and Prettier with Oxlint and Oxfmt, removing .eslintrc.json, .prettierrc, and .prettierignore, and adding .oxlintrc.json and .oxfmtrc.json for the new tools. Corresponding scripts in package.json were updated to use oxlint and oxfmt instead of eslint and prettier. [1] [2] [3] [4] [5] [6] [7] [8]

  • Updated dev dependencies in package.json to remove ESLint and Prettier related packages and add oxlint and oxfmt.

Dependency updates:

  • Updated several dependencies, including @patterninc/react-ui from ^5.0.3 to ^5.0.8 and next from ^16.1.6 to ^16.2.9.

Component and style cleanups:

  • Standardized Tailwind class order and improved readability in multiple components, such as AutoRefreshSelect, ErrorPage, Header, LeftNavBar, ClustersDetails, ClustersDetailsHeader, CommandDetails, CommandDetailsHeader, and JobDetails. [1] [2] [3] [4] [5] [6] [7] [8] [9]

  • Minor import and formatting cleanups in several files to align with new linting/formatting rules. [1] [2] [3] [4] [5] [6]

These changes streamline the development workflow by adopting faster and more modern linting and formatting tools, while also ensuring consistency and maintainability in code style and dependencies.

Copilot AI review requested due to automatic review settings July 2, 2026 09:39

Copilot AI left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

Pull request overview

This pull request migrates the web workspace from ESLint/Prettier to Oxlint/Oxfmt, updates related scripts/config, and includes a Next.js + UI dependency bump alongside small formatting/classname cleanups across several components.

Changes:

  • Replace ESLint/Prettier configuration with .oxlintrc.json / .oxfmtrc.json and update web/package.json scripts/deps accordingly.
  • Upgrade key web dependencies (notably next and @patterninc/react-ui).
  • Apply minor formatting/Tailwind class ordering and whitespace cleanups across UI components and CSS.

Reviewed changes

Copilot reviewed 19 out of 19 changed files in this pull request and generated 1 comment.

Show a summary per file
File Description
web/package.json Switch scripts to oxlint/oxfmt and update deps (Next.js / react-ui / tooling).
web/.oxlintrc.json Add Oxlint configuration (plugins, categories, ignore patterns).
web/.oxfmtrc.json Add Oxfmt configuration (quotes, width, tailwind sorting).
web/eslint.config.mjs Remove ESLint flat config.
web/.eslintrc.json Remove legacy ESLint config.
web/.prettierrc Remove Prettier config.
web/.prettierignore Remove Prettier ignore file.
web/src/common/ClientLayout/ClientLayout.tsx Formatting-only adjustments to dynamic import options.
web/src/common/AutoRefreshSelect/AutoRefreshSelect.tsx Minor JSX formatting + Tailwind class ordering.
web/src/app/globals.css Remove stray whitespace line.
web/src/components/Header/Header.tsx Normalize quoting/className ordering.
web/src/components/ErrorPage/ErrorPage.tsx Tailwind class ordering cleanup.
web/src/components/NavigationBar/LeftNavBar.tsx Remove stray whitespace and reorder Tailwind classes.
web/src/modules/Commands/CommandDetails/CommandInformationPane.tsx Remove stray whitespace in imports.
web/src/modules/Commands/CommandDetails/CommandDetails.tsx Tailwind class ordering cleanup.
web/src/modules/Commands/CommandDetails/CommandDetailsHeader.tsx Tailwind class ordering/formatting cleanup.
web/src/modules/Clusters/ClustersDetails/ClustersDetails.tsx Tailwind class ordering cleanup.
web/src/modules/Clusters/ClustersDetails/ClustersDetailsHeader.tsx Tailwind class ordering/formatting cleanup.
web/src/modules/Jobs/JobDetails/JobDetails.tsx Tailwind class ordering cleanup.

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

Comment thread web/package.json
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