cli-microsoft365 icon indicating copy to clipboard operation
cli-microsoft365 copied to clipboard

Replaces deprecated methods

Open MartinM85 opened this issue 1 year ago • 10 comments

Closes #6237

MartinM85 avatar Aug 13 '24 13:08 MartinM85

Hi @milanholemans, could you please re-run the workflow? Seems like a random error. Thanks

MartinM85 avatar Aug 14 '24 06:08 MartinM85

Thank you @MartinM85, we'll try to review it ASAP.

milanholemans avatar Aug 14 '24 06:08 milanholemans

Not sure right now, but the error on macos

FATAL ERROR: Ineffective mark-compacts near heap limit Allocation failed - JavaScript heap out of memory

Can be related to eslint-plugin-deprecation https://github.com/gund/eslint-plugin-deprecation/issues/67

MartinM85 avatar Aug 14 '24 07:08 MartinM85

Not sure right now, but the error on macos

FATAL ERROR: Ineffective mark-compacts near heap limit Allocation failed - JavaScript heap out of memory

Can be related to eslint-plugin-deprecation gund/eslint-plugin-deprecation#67

Hmm that's quite problematic.

milanholemans avatar Aug 14 '24 07:08 milanholemans

If it's failing our pipelines, and we can't fix it, we should consider something else.

milanholemans avatar Aug 14 '24 21:08 milanholemans

I have also encountered an issue with increased execution time of the linting. The CLI for M365 has a large code base, the performance of this plugin is really bad.

In the worst case, we can get rid of the plugin eslint-plugin-deprecation and keep the refactored code.

MartinM85 avatar Aug 15 '24 06:08 MartinM85

I have also encountered an issue with increased execution time of the linting. The CLI for M365 has a large code base, the performance of this plugin is really bad.

In the worst case, we can get rid of the plugin eslint-plugin-deprecation and keep the refactored code.

That's a pity. Like you mentioned, if we can't get it to work, we'll just review the refactored code.

milanholemans avatar Aug 15 '24 10:08 milanholemans

Plugin eslint-plugin-deprecation removed

MartinM85 avatar Aug 15 '24 11:08 MartinM85

@waldekmastykarz eslint-plugin-deprecation was causing the error when running test on macos.

FATAL ERROR: Ineffective mark-compacts near heap limit Allocation failed - JavaScript heap out of memory

Similar to https://github.com/gund/eslint-plugin-deprecation/issues/67

https://github.com/pnp/cli-microsoft365/actions/runs/10370949281/job/28746774555

and increased execution time of the linting, because the CLI for M365 has a large code base, the performance of this plugin is really bad.

I only refactored deprecated functions.

MartinM85 avatar Sep 15 '24 09:09 MartinM85

Thanks for clarifying. Let's update the PR's title to communicate this more clearly.

waldekmastykarz avatar Sep 15 '24 13:09 waldekmastykarz

Merged manually. Thanks for the fix!

milanholemans avatar Nov 24 '24 21:11 milanholemans