load function is marked deprecated with message saying to use function returned by load

[mike-lang]

Hi,

I believe c370f4e marked the exported load function as deprecated in error. It looks like the documentation was simply copied from the export of a Cheerio constructor over to the load function, including a deprecation message that made sense if someone was using the constructor, but is confusing when load is used.

As an example the following code:

let $ = cheerio.load(pageHTML);

produces the following JSDoc documentation in code editors:

The default cheerio instance
@deprecated - Use the function returned by load instead.

As best I can tell, this example uses the same method of loading html as is what is shown in the project documentation.

If this was not intentional, could we update the JSDoc to remove this deprecation warning and maybe update the function description?

If it is, could the project documentation we updated to indicated what is now the preferred method of loading and parsing HTML?

[fb55]

You are likely importing the default export:

import cheerio from 'cheerio';

This will import a pre-loaded instance of cheerio, which can lead to subtle bugs and issues. Instead, try

import * as cheerio from 'cheerio'

or

import { load } from 'cheerio'

Hope this helps!

[fb55]

Re-opening this as the default export should probably be removed.

[sthota-fms]

Any progress on this one?

[fb55]

See my response above.