NO-ISSUE - Security upgrades#94
Merged
Merged
Conversation
… jest/babel issues after that
- Upgrade Jest ecosystem (v28 → v30) - jest: v28.1.1 → v30.4.2 - @types/jest: v28.1.3 → v30.0.0 - ts-jest: v28.0.5 → v29.4.11 - Update jest.config.ts for ts-jest v29 compatibility - Replace deprecated toThrowError with toThrow matcher - Upgrade TypeScript (v4.7 → v5.9) - typescript: v4.7.4 → v5.9.3 - Add isolatedModules to tsconfig.json - Upgrade ESLint and TypeScript-ESLint (v5/v8 → v9/v8) - eslint: v8.57.1 → v9.39.4 - @typescript-eslint/eslint-plugin: v5.62.0 → v8.60.0 - @typescript-eslint/parser: v5.62.0 → v8.60.0 - Migrate to ESLint v9 flat config format - Remove deprecated .eslintrc and .eslintignore files - Create new eslint.config.js with flat config - Fix @octokit/rest ES module import issue - Add module mapper in jest.config.ts - Create mock implementation in test/__mocks__/@octokit/rest.ts - Code cleanup - Fix unused catch parameters in test files - Update all tests to pass with new versions All tests passing (161/161) with no lint errors.
- Upgrade express: v4.22.2 → v5.2.1 - Upgrade @types/express: v4.17.25 → v5.0.6 Breaking changes fixed: - Update wildcard route parameter syntax in archive-server.ts - Replace path parameter pattern ':path(*)' with regex /^\/download\/([^/]+)\/(.+)$/ - Express v5 uses newer path-to-regexp with stricter syntax requirements All tests passing (161/161) with no lint errors.
- Upgrade prettier: v2.8.8 → v3.8.3 - Upgrade husky: v8.0.3 → v9.1.7 - Upgrade @vercel/ncc: v0.34.0 → v0.38.4 - Upgrade fs-extra: v10.1.0 → v11.3.5 - Upgrade @types/fs-extra: v9.0.13 → v11.0.4 Breaking changes fixed: - Update src/index.ts to use 'export type' for type-only exports - Required by TypeScript isolatedModules flag All tests passing (161/161) with no lint errors.
- Rename eslint.config.js to eslint.config.mjs - Eliminates Node.js warning about module type detection - ESLint v9 automatically detects .mjs extension as ES module
fix: husky upgrade didn't remove outdated install cmd
- Updated mock to match new API signature with optional return properties - Enhanced mock to actually copy files to artifact store - Added ARTIFACT_STORE environment variable to ArchiveArtifactsMocker - Changed test ports from 3435/3436 to 3437/3438 to avoid port conflicts - All tests now passing
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
1 participant
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.
No description provided.