Skip to content
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.

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

[@rushstack/eslint-patch] v1.5.0 - The "id" argument must be of type string. Received undefined #4368

Closed
aravind25dec opened this issue Sep 28, 2023 · 9 comments · Fixed by #4374
Labels
bug Something isn't working as intended

Comments

@aravind25dec
Copy link

Post upgrade of v1.5.0 we are getting the below error in our project, it was working with 1.3.3
Oops! Something went wrong! :(

ESLint: 7.11.0

TypeError [ERR_INVALID_ARG_TYPE]: Cannot read config file: C:\Sites\xxx\xxxx\xxxx\eslint-config\index.js
Error: The "id" argument must be of type string. Received undefined
Referenced from: C:\Sites\xxxx\xxxx.eslintrc.js
at new NodeError (node:internal/errors:387:5)
at validateString (node:internal/validators:162:11)
at Module.require (node:internal/modules/cjs/loader:1091:3)
at require (C:\Sites\xxx\xxx\xxx\node_modules\v8-compile-cache\v8-compile-cache.js:159:20)
at Object. (C:\Sites\xxx\xxxx\xxxx\node_modules@rushstack\eslint-patch\lib_patch-base.js:180:55)
at Module._compile (C:\Sites\xxxxx\Web.Ui\node_modules\v8-compile-cache\v8-compile-cache.js:192:30)
at Object.Module._extensions..js (node:internal/modules/cjs/loader:1250:10)
at Module.load (node:internal/modules/cjs/loader:1074:32)
at Function.Module._load (node:internal/modules/cjs/loader:909:12)
at Module.require (node:internal/modules/cjs/loader:1098:19)

@octogonz
Copy link
Collaborator

Could you test v1.4.0? It would be super helpful to identify which PR introduced this regression.

@octogonz octogonz added the bug Something isn't working as intended label Sep 30, 2023
@octogonz
Copy link
Collaborator

@D4N14L could this be a regression from #4353?

@aravind25dec
Copy link
Author

@octogonz , its perfect v1.4.0 works so good, and I get the expected warnings. I hope the issue is with latest version

✖ 339 problems (0 errors, 339 warnings)

[email protected] lint:stylelint
stylelint "**/*.{css,scss,sass}" --allow-empty-input --config .stylelintrc.js

[email protected] lint:ts
tsc --noEmit --project .

@octogonz
Copy link
Collaborator

@D4N14L is fixing it in #4374

@aravind25dec
Copy link
Author

Hi @octogonz, will there be a new version launching with this fix?

@octogonz
Copy link
Collaborator

Yes, it automatically publishes on weekdays, but if you need it sooner we can trigger a release manually today

@aravind25dec
Copy link
Author

If it’s possible please trigger manually and let me know the new version

@octogonz
Copy link
Collaborator

octogonz commented Oct 1, 2023

@aravind25dec it's published now as version 1.5.1 🚀

@aravind25dec
Copy link
Author

thank you so much @octogonz

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug Something isn't working as intended
Projects
Archived in project
Development

Successfully merging a pull request may close this issue.

2 participants