Update querySelector type definition to return null

[johnnyoshika]

The TypeScript return type for querySelector() is HTMLElement instead of HTMLElement | null. querySelector() can potentially return null if no elements are found. This will allow code that could throw a null reference exception to pass the type checker.

Relevant type definition:

    /**
     * Query CSS Selector to find matching node.
     * @param  {string}         selector Simplified CSS selector
     * @return {HTMLElement}    matching node
     */
    querySelector(selector: string): HTMLElement;

node-html-parser/src/nodes/html.ts

Lines 383 to 386 in 2077d42

	public querySelector(selector: string) {
	return selectOne(selector, this as HTMLElement, {
	xmlMode: true,
	adapter: Matcher

Can you set the return type to HTMLElement | null?

[nonara]

Good catch! Thanks for the heads up, Johnny.

[johnnyoshika]

@nonara Thank you for the great library! I depend on it extensively in my project.

[nonara]

Happy to help! @taoqf is the library's author, but I'm sure he'll be glad to hear that it's working well for you!

I have this and several other fixes in place in the following PR, but I'll also follow-up in this thread when we have a release ready.

• Multiple fixes #161

[nonara]

Corrected in v5