chore(config): Update development tooling and dependencies#13
Merged
rubenszinho merged 3 commits intodevelopfrom May 15, 2025
Merged
chore(config): Update development tooling and dependencies#13rubenszinho merged 3 commits intodevelopfrom
rubenszinho merged 3 commits intodevelopfrom
Conversation
There was a problem hiding this comment.
Pull Request Overview
This PR updates the development configuration by modernizing dependencies, refining build processes, and introducing improvements to ESLint and import aliasing.
- Adds path alias configuration and ESM __dirname support in vite.config.ts
- Updates dependencies and scripts in package.json to support new build modes and development tooling
- Implements a TypeScript-specific override in eslint.config.js to disable no-undef
Reviewed Changes
Copilot reviewed 4 out of 4 changed files in this pull request and generated 1 comment.
| File | Description |
|---|---|
| vite.config.ts | Introduced path aliasing and ESM __dirname support with HMR improvements |
| public/mockServiceWorker.js | Updated mocked service worker version for consistency |
| package.json | Adjusted version, scripts, and dependencies to modernize dev tooling |
| eslint.config.js | Added an override for TypeScript to disable no-undef as advised by TS ESLint |
Comments suppressed due to low confidence (1)
package.json:4
- Confirm that setting the package version to '0.0.0' is intentional for development purposes, as it may lead to confusion if deployed accidentally.
"version": "0.0.0",
Comment on lines
+27
to
+28
| // This enables HMR for i18n JSON files | ||
| usePolling: true, |
There was a problem hiding this comment.
Using file polling for HMR can impact overall performance; consider limiting polling to specific scenarios or validating its necessity in your development setup.
Suggested change
| // This enables HMR for i18n JSON files | |
| usePolling: true, | |
| // Enable polling only for i18n JSON files to minimize performance impact | |
| usePolling: true, | |
| ignored: ['**/node_modules/**', '**/dist/**', '**/src/**', '!**/src/i18n/locales/**/*.json'], |
186b1bd to
ea550bb
Compare
rubenszinho
added a commit
that referenced
this pull request
May 29, 2025
* chore: Update development tooling and dependencies * chore: generate lock * chore: update react-helmet dependency
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
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
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.
This PR updates the project's development configuration, improves build processes, and modernizes dependencies. It includes ESLint improvements, path aliasing, and additional build modes to enhance developer experience.
✨ Changes Introduced