-
Notifications
You must be signed in to change notification settings - Fork 21
Bump vite #1384
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Merged
MykolaGolubyev
merged 33 commits into
testingisdocumenting:master
from
antoinell:bump-vite
Dec 9, 2025
Merged
Bump vite #1384
Changes from all commits
Commits
Show all changes
33 commits
Select commit
Hold shift + click to select a range
46d9940
bump mermaid to 11.12.1
antoinell 737c873
change initialize method call in order not to call the deprecated API
antoinell 2bdc3dd
bumping vite and node too
antoinell 38153c3
updating other plugins related to vite
antoinell ab0a76e
further changes
antoinell 4927b6c
changing configuration to make it more plain vanilla
antoinell f08e88f
trying to improve
antoinell 946dec1
removing jsx pragma
antoinell 8734f66
Merge branch 'master' into bump-mermaid
antoinell ae5a5a9
rolling back to react 17
antoinell 5178dd2
changes to deploy more javascript files
antoinell 6efab3d
fixed error the javascript files generated by vite should be loaded a…
antoinell b1f41ba
readding the license headers.
antoinell e7c1e71
restoring SearchBox and SearchToc
antoinell 1ffe26d
restoring SearchTocItem
antoinell b5e0379
trying to get npm run dev to work, not there yet
antoinell 657b558
changing the eslint code
antoinell 50adc71
executing npx eslint . --fix
antoinell 045bb0e
changing code related to loading the index so that we do not load the…
antoinell c58f45d
rolling back to mermaid 8.9.2
antoinell 8f1a7a4
rolling back to mermaid 8.9.2
antoinell da271d3
making sure that `npm run dev` makes vite display the port number
antoinell 2bd20fa
removing file indexCreation.js since we are not using it anymore.
antoinell 8124f25
Merge branch 'master' into bump-vite
antoinell 4fcaad3
Merge branch 'master' into bump-vite
antoinell b23bdc7
Add a line ` * Copyright 2025 znai maintainers` before the existing T…
antoinell d27e51e
Remove mutation observer.
antoinell f37ec69
Remove main.js
antoinell e02e956
use import type in CliCommand.tsx.
antoinell 14ede3b
removing // @ts-ignore
antoinell 2de9fba
remove @ts-ignore annotation
antoinell 065815c
remove @ts-ignore annotation
antoinell 040d099
Merge branch 'master' into bump-vite
antoinell File filter
Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
There are no files selected for viewing
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This file was deleted.
Oops, something went wrong.
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
| Original file line number | Diff line number | Diff line change |
|---|---|---|
| @@ -0,0 +1,73 @@ | ||
| import js from '@eslint/js'; | ||
|
Member
There was a problem hiding this comment. Choose a reason for hiding this commentThe reason will be displayed to describe this comment to others. Learn more. is this file required for Vite migration?
Contributor
Author
There was a problem hiding this comment. Choose a reason for hiding this commentThe reason will be displayed to describe this comment to others. Learn more. The new eslint consumes this file |
||
| import globals from 'globals'; | ||
| import react from 'eslint-plugin-react'; | ||
| import reactHooks from 'eslint-plugin-react-hooks'; | ||
| import reactRefresh from 'eslint-plugin-react-refresh'; | ||
| import typescriptEslint from '@typescript-eslint/eslint-plugin'; | ||
| import parser from '@typescript-eslint/parser'; | ||
|
|
||
| export default [ | ||
| // Ignore patterns | ||
| { | ||
| ignores: ['dist', 'build', 'node_modules', '*.config.js'], | ||
| }, | ||
|
|
||
| // Base JavaScript configuration | ||
| { | ||
| files: ['**/*.{js,jsx,mjs,cjs,ts,tsx}'], | ||
| languageOptions: { | ||
| parser: parser, | ||
| ecmaVersion: 2020, | ||
| globals: { | ||
| ...globals.browser, | ||
| ...globals.es2020, | ||
| }, | ||
| parserOptions: { | ||
| ecmaVersion: 'latest', | ||
| ecmaFeatures: { jsx: true }, | ||
| sourceType: 'module', | ||
| }, | ||
| }, | ||
| settings: { | ||
| react: { | ||
| version: 'detect', | ||
| }, | ||
| }, | ||
| plugins: { | ||
| react, | ||
| 'react-hooks': reactHooks, | ||
| 'react-refresh': reactRefresh, | ||
| '@typescript-eslint': typescriptEslint, | ||
| }, | ||
| rules: { | ||
| // ESLint recommended rules | ||
| '@typescript-eslint/consistent-type-imports': [ | ||
| 'error', | ||
| { | ||
| prefer: 'type-imports', | ||
| fixStyle: 'separate-type-imports', | ||
| }, | ||
| ], | ||
| ...js.configs.recommended.rules, | ||
|
|
||
| // React rules | ||
| ...react.configs.recommended.rules, | ||
| ...react.configs['jsx-runtime'].rules, | ||
| 'react/jsx-no-target-blank': 'off', | ||
| 'react/prop-types': 'off', // Turn off if using TypeScript | ||
|
|
||
| // React Hooks rules | ||
| ...reactHooks.configs.recommended.rules, | ||
|
|
||
| // React Refresh | ||
| 'react-refresh/only-export-components': [ | ||
| 'warn', | ||
| { allowConstantExport: true }, | ||
| ], | ||
|
|
||
| // Custom rules | ||
| 'no-unused-vars': ['warn', { argsIgnorePattern: '^_' }], | ||
| 'no-console': ['warn', { allow: ['warn', 'error'] }], | ||
| }, | ||
| }, | ||
| ]; | ||
Oops, something went wrong.
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
where did you move indexCreation.js?
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
It is replaced by another file called initialization.js in another package.