Skip to content

Convert to vite native tooling#2118

Open
ejsmith wants to merge 1 commit intomainfrom
vite-native
Open

Convert to vite native tooling#2118
ejsmith wants to merge 1 commit intomainfrom
vite-native

Conversation

@ejsmith
Copy link
Member

@ejsmith ejsmith commented Feb 17, 2026

This pull request introduces Oxlint and Oxfmt into the project's frontend workflow to improve code quality and formatting consistency. It updates configuration files, scripts, and dependencies to integrate these tools alongside existing linting and formatting solutions. The changes also adjust the ESLint and Vite configurations to support Oxlint, and update the package.json to use the latest rolldown-vite build.

Tooling and Linting Integration:

  • Added .oxlintrc.json and .oxfmtrc.json configuration files to define project-specific rules for Oxlint and Oxfmt. [1] [2]
  • Updated eslint.config.js to include eslint-plugin-oxlint and to load Oxlint configuration, ensuring ESLint rules do not overlap with Oxlint. [1] [2]
  • Integrated vite-plugin-oxlint into the Vite configuration, enabling Oxlint checks during the build process. [1] [2]

Scripts and Dependency Updates:

  • Modified package.json scripts to add lint:oxfmt, lint:oxlint, and corresponding format commands, and to ensure Oxlint and Oxfmt are run as part of the main linting and formatting workflows.
  • Added eslint-plugin-oxlint, vite-plugin-oxlint, oxlint, and oxfmt to devDependencies.
  • Switched the vite dependency to use the latest rolldown-vite build and updated the overrides section accordingly.

These changes help enforce stricter code standards and ensure a more consistent codebase by leveraging the latest linting and formatting tools.

@github-actions
Copy link

Code Coverage

Package Line Rate Branch Rate Complexity Health
Exceptionless.Insulation 24% 23% 208
Exceptionless.Core 66% 59% 7446
Exceptionless.AppHost 26% 14% 55
Exceptionless.Web 55% 42% 3489
Summary 61% (11865 / 19532) 53% (5655 / 10706) 11198

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Development

Successfully merging this pull request may close these issues.

1 participant