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

Added content to CI section on htmlproofer #93

Merged
merged 8 commits into from
Sep 30, 2024

Conversation

kierisi
Copy link
Collaborator

@kierisi kierisi commented Sep 19, 2024

based on review feedback in #92 (originally given in #79), I've added a first pass at information on htmlproofer. a couple of notes:

  • the HTMLProofer documentation uses HTMLProofer to refer to the software, and htmlproofer when it's being used within github. I don't know if we want to keep those conventions or not, as I seem to recall we default to 'htmlproofer'.
  • I tried to introduce breaking changes to get HTMLProofer to throw an error and was unsuccessful. I can file an issue in case anyone else comes across an error so that we can get a "wild-caught" screenshot, but also as a reminder to myself to keep trying to get htmlproofer to break. nevermind, I actually managed to introduce a bunch of breaks that tripped up htmlproofer and got me the images I needed!

@kierisi kierisi marked this pull request as draft September 19, 2024 22:05
@kierisi
Copy link
Collaborator Author

kierisi commented Sep 19, 2024

pre-commit.ci autofix

@kierisi kierisi marked this pull request as ready for review September 19, 2024 22:23
@kierisi
Copy link
Collaborator Author

kierisi commented Sep 19, 2024

@lwasser this is also ready for review! there are six new files: 5 images, 1 text. the text goes through the addition of htmlproofer to the CI page of the handbook, as suggested in #92.

@lwasser
Copy link
Member

lwasser commented Sep 30, 2024

pre-commit.ci autofix

@lwasser
Copy link
Member

lwasser commented Sep 30, 2024

@kierisi awesome! thanks! i made some edits and will merge once CI is happy again.

@lwasser lwasser merged commit 6ec0d48 into pyOpenSci:main Sep 30, 2024
3 checks passed
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 this pull request may close these issues.

2 participants