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

Updates dependencies #6416

Closed
wants to merge 1 commit into from
Closed

Conversation

waldekmastykarz
Copy link
Member

Updates dependencies. Here's what's left:

Package                            Current    Wanted  Latest  Location                                       Depended by
@types/node                       20.16.11  20.16.11  22.7.5  node_modules/@types/node                       cli-microsoft365
@typescript-eslint/eslint-plugin    7.18.0    7.18.0   8.8.1  node_modules/@typescript-eslint/eslint-plugin  cli-microsoft365
@typescript-eslint/parser           7.18.0    7.18.0   8.8.1  node_modules/@typescript-eslint/parser         cli-microsoft365
adaptivecards                        3.0.4     3.0.4   1.3.1  node_modules/adaptivecards                     cli-microsoft365
eslint                              8.57.0    8.57.1  9.12.0  node_modules/eslint                            cli-microsoft365

To upgrade eslint, we need to migrate our config to a new format.
To migrate @typescript-eslint packages, we need to address two missing rules that are no longer present in the new version. I suggest we create separate issues for both tasks.

@milanholemans
Copy link
Contributor

We already have #6350 for this. Not sure if migrating typescript-eslint packages is part of this.

@milanholemans milanholemans added this to the v10 milestone Oct 17, 2024
@milanholemans
Copy link
Contributor

Merged manually, thanks!

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
pr-merged pr-priority Process this PR asap
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants