Update @woocommerce/eslint-plugin to the latest 1.3

[asumaran]

Hi @asumaran @ricardo , sorry for the late reply! I am good with the // eslint-disable-next-line as well. 👍

I think the idea was to prevent the use of listener on window document globals inside hooks and react components. It looks like the no global rule is removed in WordPress/gutenberg#34528. Maybe we can try updating our @woocommerce/eslint-plugin to the latest 1.3?

Originally posted by @harriswong in #2252 (comment)

[asumaran]

The idea to update the @woocommerce/eslint-plugin dependency in our project was to remove the "no global event" rule that was added recently. The PR that removes it has been merged but has not been included in the @wordpress/eslint-plugin@8.x.x package.

Unfortunately, @woocommerce/eslint-plugin is still using the 8.x.x series of the @wordpress/eslint-plugin package as a dependency. The rule is still present in the 8.0.2 version of the @wordpress/eslint-plugin.

We need to wether update @woocommerce/eslint-plugin to use the latest @wordpress/eslint-plugin or merge WordPress/gutenberg#34528 into the 8.x.x version of the @wordpress/eslint-plugin.

[asumaran]

@woocommerce/eslint-plugin updated @wordpress/eslint-plugin from v8.0.0 to v11.0.0 in woocommerce/woocommerce-admin#8475 but a new release of the 1.3.x branch of the @woocommerce/eslint-plugin extension wasn't released up until today.

1.3.x seems abandoned so probably we should update @woocommerce/eslint-plugin to its latest version instead (v2.2.0)