-
Notifications
You must be signed in to change notification settings - Fork 256
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
[all hosts] (Developer Experience) Adjust lint configuration for eslint v9 #5006
Conversation
Learn Build status updates of commit 571dd7f: ✅ Validation status: passed
For more details, please refer to the build report. For any questions, please:
|
] | ||
} | ||
1. In the root of the project, create a text file named **eslint.config.js** (or **.mjs**), if there isn't one already there. Be sure to inherit the recommended configuration for `eslint-plugin-office-addins`. The `plugins` array should include `eslint-plugin-office-addins`. The following is a simple example that includes settings for TypeScript. Your **eslint.config.js** file may have additional properties and configurations. | ||
|
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Technically plugins
is not an array anymore . . . it is an object.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Good catch! Corrected.
Learn Build status updates of commit 8de9973: ✅ Validation status: passed
For more details, please refer to the build report. For any questions, please:
|
@millerds, since your upstream changes were checked in, I'll merge this PR by EOD. Please let me know if there are any objections. |
Based on upstream changes to the Office Add-ins tooling for the linter: OfficeDev/Office-Addin-Scripts#917