Skip to content

fix(eslint-plugin-fiori-tools): fix heap OOM on macos Node 20 arm64 #4570

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.

Sign up for GitHub

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
longieirl merged 12 commits into from
Apr 23, 2026
+3 −47
Merged

fix(eslint-plugin-fiori-tools): fix heap OOM on macos Node 20 arm64 #4570

Show file tree
Hide file tree
Changes from all commits
Commits
Show all changes
12 commits
Select commit Hold shift + click to select a range
f29244c
fix(eslint-plugin-fiori-tools): fix heap OOM by switching to jest cov…
longieirl Apr 17, 2026
8e6b963
Merge origin/main into fix/4545-eslint-plugin-oom-v2
longieirl Apr 17, 2026
f427d4e
chore: update pnpm-lock.yaml after merging main
longieirl Apr 17, 2026
f01d837
Merge branch 'main' into fix/4545-eslint-plugin-oom-v2
longieirl Apr 20, 2026
e7ac523
Delete .changeset/eslint-plugin-fiori-tools-oom-fix-v2.md
longieirl Apr 21, 2026
2d7414f
Merge branch 'main' into fix/4545-eslint-plugin-oom-v2
longieirl Apr 21, 2026
b2c7348
Merge branch 'main' into fix/4545-eslint-plugin-oom-v2
longieirl Apr 22, 2026
1d66814
Merge branch 'main' into fix/4545-eslint-plugin-oom-v2
longieirl Apr 22, 2026
9db177b
Merge branch 'main' into fix/4545-eslint-plugin-oom-v2
longieirl Apr 22, 2026
bb02e3e
Merge branch 'main' into fix/4545-eslint-plugin-oom-v2
longieirl Apr 23, 2026
b68feeb
Merge branch 'main' into fix/4545-eslint-plugin-oom-v2
longieirl Apr 23, 2026
8e9ada3
Merge branch 'main' into fix/4545-eslint-plugin-oom-v2
longieirl Apr 23, 2026
File filter

Filter by extension

Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
4 changes: 2 additions & 2 deletions packages/eslint-plugin-fiori-tools/jest.config.js
View file
Open in desktop
Original file line number Diff line number Diff line change
@@ -1,7 +1,7 @@
const config = require('../../jest.base');
module.exports = {
...config,
// Coverage is handled by c8 wrapper for worker thread support
collectCoverage: false,
// coverageProvider: 'v8' overrides jest.base default ('babel'); collectCoverage: true is inherited from jest.base
coverageProvider: 'v8',
Comment thread
longieirl marked this conversation as resolved.
setupFiles: ['<rootDir>/test/global-setup.ts']
}
3 changes: 1 addition & 2 deletions packages/eslint-plugin-fiori-tools/package.json
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -20,10 +20,9 @@
"format": "prettier --write '**/*.{js,json,ts,yaml,yml}' --ignore-path ../../.prettierignore",
"lint": "eslint",
"lint:fix": "eslint --fix",
"test": "c8 --reporter=text --reporter=lcov --merge-async --exclude='test/**' --exclude='**/*.test.ts' --include='src/**/*.ts' cross-env NODE_OPTIONS='--experimental-vm-modules' jest --ci --forceExit --detectOpenHandles --colors --maxWorkers=2"
Comment thread
heimwege marked this conversation as resolved.
"test": "cross-env NODE_OPTIONS='--experimental-vm-modules' jest --ci --forceExit --detectOpenHandles --colors --maxWorkers=50%"
},
"devDependencies": {
"c8": "^11.0.0",
"cross-env": "10.1.0",
"eslint": "9.39.1",
"@typescript-eslint/rule-tester": "8.57.2",
Expand Down
43 changes: 0 additions & 43 deletions pnpm-lock.yaml
View file
Open in desktop

Some generated files are not rendered by default. Learn more about how customized files appear on GitHub.

Loading