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

Enable MD045 rule #335

Closed
wants to merge 3 commits into from
Closed

Enable MD045 rule #335

wants to merge 3 commits into from

Conversation

XhmikosR
Copy link
Contributor

@XhmikosR XhmikosR commented Jun 7, 2020

Images should have alternate text (alt text)

Refs #256.

Images should have alternate text (alt text)
@netlify
Copy link

netlify bot commented Jun 7, 2020

Deploy preview for pihole-docs ready!

Built with commit b0f5f08

https://deploy-preview-335--pihole-docs.netlify.app

@dschaper
Copy link
Member

dschaper commented Jul 4, 2020

Should we fix the missing alt text in a PR first before flipping this out of draft status?

@XhmikosR
Copy link
Contributor Author

XhmikosR commented Jul 6, 2020

Should we fix the missing alt text in a PR first before flipping this out of draft status?

Yup. That's why I still have this as a draft :)

@dschaper
Copy link
Member

dschaper commented Jul 6, 2020

The instructions were not clear so no one else has made any PRs to clean things up. Now that we know the images need to be fixed we can get things moving.

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