Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Error: undefined method `xpath' for nil:NilClass #544

Closed
nickoe opened this issue Nov 17, 2019 · 2 comments · Fixed by #545
Closed

Error: undefined method `xpath' for nil:NilClass #544

nickoe opened this issue Nov 17, 2019 · 2 comments · Fixed by #545

Comments

@nickoe
Copy link

nickoe commented Nov 17, 2019

With the latest version of htlm-proofer 3.14.0 I get this error for my site. For example my command and output is as follows:

htmlproofer ./public --external_only --only_4xx --url-ignore "/www.digikey.com/"
Running ["LinkCheck", "ScriptCheck", "ImageCheck"] on ["./public"] on *.html... 
htmlproofer 3.14.0 | Error:  undefined method `xpath' for nil:NilClass
The command "htmlproofer ./public --external_only --only_4xx --url-ignore "/www.digikey.com/"" exited with 1.
Done. Your build exited with 1.

https://travis-ci.org/KiCad/kicad-website/builds/613212365

nickoe added a commit to KiCad/kicad-website that referenced this issue Nov 17, 2019
nickoe added a commit to KiCad/kicad-website that referenced this issue Nov 17, 2019
This is to workaround and error we were getting:
Error: undefined method `xpath' for nil:NilClass

Reported at:
gjtorikian/html-proofer#544
nickoe added a commit to KiCad/kicad-website that referenced this issue Nov 17, 2019
This is to workaround and error we were getting:
Error: undefined method `xpath' for nil:NilClass

Reported at:
gjtorikian/html-proofer#544
@gjtorikian
Copy link
Owner

Sorry about that, there was a typo in one of the methods being used. A bugfix will be pushed shortly.

@nickoe
Copy link
Author

nickoe commented Nov 18, 2019

@gjtorikian Thank you for the quick fix and release. I will go back to using latest release again.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging a pull request may close this issue.

2 participants