You signed in with another tab or window. Reload to refresh your session.You signed out in another tab or window. Reload to refresh your session.You switched accounts on another tab or window. Reload to refresh your session.Dismiss alert
Linting a .crx file that is in CRX3 (rather than CRX2) format fails.
Anything else we should know?
This stems from the package dependency on crx-parser. I've submitted a PR that fixes this issue (shyiko/node-crx-parser#2) but considering the library hasn't been published in 4 years I'm not optimistic that it'll get accepted and published.
If there's not a response from the original package maintainer, I'm considering publishing my fork to npm and making a PR to addons-linter to change the dependency, but wanted to get feedback from the addons-linter community before going that route.
The text was updated successfully, but these errors were encountered:
Describe the problem and steps to reproduce it:
Linting a .crx file that is in CRX3 (rather than CRX2) format fails.
Anything else we should know?
This stems from the package dependency on
crx-parser
. I've submitted a PR that fixes this issue (shyiko/node-crx-parser#2) but considering the library hasn't been published in 4 years I'm not optimistic that it'll get accepted and published.If there's not a response from the original package maintainer, I'm considering publishing my fork to npm and making a PR to addons-linter to change the dependency, but wanted to get feedback from the addons-linter community before going that route.
The text was updated successfully, but these errors were encountered: