Feature/dependency updates#9
Conversation
- Upgraded @babel/core from ^7.27.4 to ^7.28.4 - Upgraded @babel/preset-env from ^7.27.2 to ^7.28.3 - Upgraded @eslint/js from ^9.33.0 to ^9.37.0 - Upgraded @stryker-mutator/core from ^9.0.1 to ^9.2.0 - Upgraded @stryker-mutator/jest-runner from ^9.0.1 to ^9.2.0 - Upgraded babel-jest from ^30.0.0-beta.3 to ^30.2.0 - Upgraded dependency-cruiser from ^17.0.1 to ^17.0.2 - Upgraded eslint from ^9.33.0 to ^9.37.0 - Upgraded fast-check from ^4.2.0 to ^4.3.0 - Upgraded globals from ^16.3.0 to ^16.4.0 - Upgraded jest-environment-jsdom from ^30.0.5 to ^30.2.0
Summary of ChangesHello @cs-util, I'm Gemini Code Assist1! I'm currently reviewing this pull request and will post my feedback shortly. In the meantime, here's a summary to help you and other reviewers quickly get up to speed! This pull request focuses on enhancing the project's dependency management and development tooling. It updates a range of development dependencies to their latest versions, introduces a new dependency for styling, and refines the code formatting workflow to ensure consistency across configuration files. Additionally, it reinforces best practices for maintaining dependency health and security through updated development documentation. Highlights
Using Gemini Code AssistThe full guide for Gemini Code Assist can be found on our documentation page, here are some quick tips. Invoking Gemini You can request assistance from Gemini at any point by creating a comment using either
Customization To customize Gemini Code Assist for GitHub experience, repository maintainers can create a configuration file and/or provide a custom code review style guide (such as PEP-8 for Python) by creating and adding files to a Limitations & Feedback Gemini Code Assist may make mistakes. Please leave feedback on any instances where its feedback is incorrect or counter productive. You can react with 👍 and 👎 on @gemini-code-assist comments. If you're interested in giving your feedback about your experience with Gemini Code Assist for Github and other Google products, sign up here. You can also get AI-powered code generation, chat, as well as code reviews directly in the IDE at no cost with the Gemini Code Assist IDE Extension. Footnotes
|
There was a problem hiding this comment.
Code Review
This pull request updates several development dependencies to their latest versions, adds tailwindcss, and improves the development workflow by including package.json in the formatting script and updating the development guidelines in AGENTS.md. The changes are well-described and improve the project's dependency management. My main feedback is to sort the devDependencies in package.json alphabetically to enhance maintainability, which aligns with the goal of improving the project's formatting standards.
| "@babel/core": "^7.28.4", | ||
| "@babel/preset-env": "^7.28.3", | ||
| "@eslint/js": "^9.37.0", | ||
| "@playwright/test": "^1.49.0", | ||
| "@stryker-mutator/core": "^9.0.1", | ||
| "@stryker-mutator/jest-runner": "^9.0.1", | ||
| "babel-jest": "^30.0.0-beta.3", | ||
| "dependency-cruiser": "^17.0.1", | ||
| "eslint": "^9.33.0", | ||
| "@stryker-mutator/core": "^9.2.0", | ||
| "@stryker-mutator/jest-runner": "^9.2.0", | ||
| "babel-jest": "^30.2.0", | ||
| "dependency-cruiser": "^17.0.2", | ||
| "eslint": "^9.37.0", | ||
| "eslint-plugin-jest": "^29.0.1", | ||
| "fast-check": "^4.2.0", | ||
| "globals": "^16.3.0", | ||
| "fast-check": "^4.3.0", | ||
| "globals": "^16.4.0", | ||
| "jest": "^29.7.0", | ||
| "jest-environment-jsdom": "^30.0.5", | ||
| "jest-environment-jsdom": "^30.2.0", | ||
| "jscpd": "^4.0.5", | ||
| "madge": "^8.0.0", | ||
| "tailwindcss": "^4.1.14", | ||
| "prettier": "^3.3.3", | ||
| "serve": "^14.2.1" |
There was a problem hiding this comment.
To improve maintainability and make it easier to find dependencies, it's a good practice to keep them alphabetically sorted. Since you've added prettier formatting for package.json, this would be a good opportunity to sort this list.
| "@babel/core": "^7.28.4", | |
| "@babel/preset-env": "^7.28.3", | |
| "@eslint/js": "^9.37.0", | |
| "@playwright/test": "^1.49.0", | |
| "@stryker-mutator/core": "^9.0.1", | |
| "@stryker-mutator/jest-runner": "^9.0.1", | |
| "babel-jest": "^30.0.0-beta.3", | |
| "dependency-cruiser": "^17.0.1", | |
| "eslint": "^9.33.0", | |
| "@stryker-mutator/core": "^9.2.0", | |
| "@stryker-mutator/jest-runner": "^9.2.0", | |
| "babel-jest": "^30.2.0", | |
| "dependency-cruiser": "^17.0.2", | |
| "eslint": "^9.37.0", | |
| "eslint-plugin-jest": "^29.0.1", | |
| "fast-check": "^4.2.0", | |
| "globals": "^16.3.0", | |
| "fast-check": "^4.3.0", | |
| "globals": "^16.4.0", | |
| "jest": "^29.7.0", | |
| "jest-environment-jsdom": "^30.0.5", | |
| "jest-environment-jsdom": "^30.2.0", | |
| "jscpd": "^4.0.5", | |
| "madge": "^8.0.0", | |
| "tailwindcss": "^4.1.14", | |
| "prettier": "^3.3.3", | |
| "serve": "^14.2.1" | |
| "@babel/core": "^7.28.4", | |
| "@babel/preset-env": "^7.28.3", | |
| "@eslint/js": "^9.37.0", | |
| "@playwright/test": "^1.49.0", | |
| "@stryker-mutator/core": "^9.2.0", | |
| "@stryker-mutator/jest-runner": "^9.2.0", | |
| "babel-jest": "^30.2.0", | |
| "dependency-cruiser": "^17.0.2", | |
| "eslint": "^9.37.0", | |
| "eslint-plugin-jest": "^29.0.1", | |
| "fast-check": "^4.3.0", | |
| "globals": "^16.4.0", | |
| "jest": "^29.7.0", | |
| "jest-environment-jsdom": "^30.2.0", | |
| "jscpd": "^4.0.5", | |
| "madge": "^8.0.0", | |
| "prettier": "^3.3.3", | |
| "serve": "^14.2.1", | |
| "tailwindcss": "^4.1.14" |
…ctoring coveredMutants and only calculating the mutation score when that count is positive. This avoids dividing by zero when every mutant is marked NoCoverage
This pull request focuses on improving dependency management practices and updating development tooling. The most significant changes include updating several devDependencies to their latest versions, adding a new dependency, and enhancing the formatting workflow to include
package.json. Additionally, the development documentation now emphasizes routine dependency checks.Dependency management and updates:
package.jsonto newer versions, including@babel/core,@babel/preset-env,@eslint/js,@stryker-mutator/core,@stryker-mutator/jest-runner,babel-jest,dependency-cruiser,eslint,fast-check,globals, andjest-environment-jsdom.tailwindcssas a new devDependency inpackage.json.AGENTS.mdto recommend runningnpm outdatedregularly and keeping dependencies up to date, with guidance on handling exceptions.Development workflow improvements:
formatscript inpackage.jsonto includepackage.jsonitself in Prettier formatting, ensuring consistent formatting for the dependency file.