Skip to content
This repository has been archived by the owner on Jul 29, 2022. It is now read-only.

Adding HTMLProofer to Travis build #28

Merged
merged 4 commits into from
Apr 23, 2021
Merged

Adding HTMLProofer to Travis build #28

merged 4 commits into from
Apr 23, 2021

Conversation

l-mb
Copy link
Contributor

@l-mb l-mb commented Apr 23, 2021

Signed-off-by: Lars Marowsky-Bree [email protected]

@l-mb l-mb force-pushed the main branch 2 times, most recently from 0bc1ece to d19c667 Compare April 23, 2021 13:39
Signed-off-by: Lars Marowsky-Bree <[email protected]>
@l-mb l-mb changed the title [WIP] [DNM] Adding HTMLProofer to Travis build Adding HTMLProofer to Travis build Apr 23, 2021
@l-mb l-mb requested a review from asettle April 23, 2021 13:47
@l-mb l-mb removed the request for review from asettle April 23, 2021 14:02
l-mb added 2 commits April 23, 2021 16:04
Signed-off-by: Lars Marowsky-Bree <[email protected]>
Signed-off-by: Lars Marowsky-Bree <[email protected]>
@l-mb l-mb requested a review from asettle April 23, 2021 14:29
@l-mb
Copy link
Contributor Author

l-mb commented Apr 23, 2021

I removed the HTML header changes because even if I slipped them into the proper HEAD element, they still broke the site formatting and I wasn't quite sure what they were trying to achieve.

I'd suggest to re-add them later.

@l-mb l-mb requested a review from s0nea April 23, 2021 14:33
@l-mb l-mb merged commit 71bf75f into aquarist-labs:main Apr 23, 2021
@s0nea
Copy link
Member

s0nea commented Apr 23, 2021

I removed the HTML header changes because even if I slipped them into the proper HEAD element, they still broke the site formatting and I wasn't quite sure what they were trying to achieve.

I'd suggest to re-add them later.

Yep, thank you! I already had an issue in mind: #29

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

Successfully merging this pull request may close these issues.

3 participants