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 1 commit
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
5 changes: 5 additions & 0 deletions .changeset/eslint-plugin-fiori-tools-oom-fix-v2.md
View file
Open in desktop
Original file line number Diff line number Diff line change
@@ -0,0 +1,5 @@
---
"@sap-ux/eslint-plugin-fiori-tools": patch
Comment thread
longieirl marked this conversation as resolved.
Outdated
---

fix(eslint-plugin-fiori-tools): fix heap OOM on macos Node 20 arm64 by switching to jest coverageProvider v8
3 changes: 1 addition & 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,6 @@
const config = require('../../jest.base');
module.exports = {
...config,
// Coverage is handled by c8 wrapper for worker thread support
collectCoverage: false,
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 --exclude='test/**' --exclude='**/*.test.ts' --include='src/**/*.ts' cross-env NODE_OPTIONS='--experimental-vm-modules' jest --ci --forceExit --detectOpenHandles --colors"
"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