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

htmlproofer 4.4.2 error in GitHub Actions #30

Closed
bmaupin opened this issue Oct 12, 2022 · 1 comment
Closed

htmlproofer 4.4.2 error in GitHub Actions #30

bmaupin opened this issue Oct 12, 2022 · 1 comment

Comments

@bmaupin
Copy link
Owner

bmaupin commented Oct 12, 2022

htmlproofer 4.4.2 | Error:  Invalid value for 'disable_external': 'true'. Expected FalseClass.

https://github.com/bmaupin/wiki/actions/runs/3229240322/jobs/5286303080

@bmaupin
Copy link
Owner Author

bmaupin commented Oct 12, 2022

Seems to be caused by this bug: gjtorikian/html-proofer#773

It was fixed in 4.4.3, but the GitHub Action we're using hasn't been upgraded, apparently because the build is failing: https://github.com/chabad360/htmlproofer/actions

There's already a bug for it: chabad360/htmlproofer#10

The creator of that action does seem to be semi-active on GitHub: https://github.com/chabad360

but ... this isn't the first time this action has broken, and there's another upcoming breaking change with HTMLProofer v5: gjtorikian/html-proofer#758

One of the biggest issues with https://github.com/chabad360/htmlproofer is that it isn't pinned to a specific version of HTMLProofer: https://github.com/chabad360/htmlproofer/blob/master/action.yml

And the container image itself just gets overwritten with the newest version, so there's no way to pin that either: https://hub.docker.com/r/chabad360/htmlproofer

At this point, I think the best path forward is to migrate to a different GitHub Action. This one has more stars: https://github.com/anishathalye/proof-html

Plus it's pinned to a specific container image: https://github.com/anishathalye/proof-html/blob/master/action.yml

And old versions of the container image are preserved: https://hub.docker.com/r/anishathalye/proof-html/tags

bmaupin added a commit that referenced this issue Oct 13, 2022
... to specify parameters that don't exist in the GitHub Action

Fixes #30
bmaupin added a commit that referenced this issue Oct 13, 2022
... to specify parameters that don't exist in the GitHub Action

Fixes #30
bmaupin added a commit that referenced this issue Oct 13, 2022
... to specify parameters that don't exist in the GitHub Action

Fixes #30
bmaupin added a commit that referenced this issue Oct 13, 2022
... to specify parameters that don't exist in the GitHub Action

Fixes #30
bmaupin added a commit that referenced this issue Oct 13, 2022
... to specify parameters that don't exist in the GitHub Action

Fixes #30
bmaupin added a commit that referenced this issue Oct 13, 2022
... to specify parameters that don't exist in the GitHub Action

Fixes #30
bmaupin added a commit that referenced this issue Mar 15, 2023
This allows us to lock it to a specific version so it doesn't break every time htmlproofer changes. Unfortunately it doesn't have all the parameters we need, but thankfully the command can be overridden.

Fixes #30
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

No branches or pull requests

1 participant