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

Add some fixes for the markdown linter #5842

Merged
merged 3 commits into from
Jan 9, 2023

Conversation

antgamdia
Copy link
Contributor

Description of the change

This PR is fixing some pending issues highlighted by the markdownlint linter. A first batch of them have been fixed by just using the --fix arg, but a bunch of them still required some manual process.

Benefits

Most of the true positive warnings will be gone; besides, the copyright year of the website will get updated to 2023 :P (it is generated based upon the last deployment date).

Possible drawbacks

Don't think so, but let's double-check with the site preview just in case.

Applicable issues

N/A

Additional information

N/A

Signed-off-by: Antonio Gamez Diaz <agamez@vmware.com>
Signed-off-by: Antonio Gamez Diaz <agamez@vmware.com>
Signed-off-by: Antonio Gamez Diaz <agamez@vmware.com>
@netlify
Copy link

netlify bot commented Jan 9, 2023

Deploy Preview for kubeapps-dev ready!

Built without sensitive environment variables

Name Link
🔨 Latest commit b328b5e
🔍 Latest deploy log https://app.netlify.com/sites/kubeapps-dev/deploys/63bc163f0f3973000aa303f2
😎 Deploy Preview https://deploy-preview-5842--kubeapps-dev.netlify.app
📱 Preview on mobile
Toggle QR Code...

QR Code

Use your smartphone camera to open QR code link.

To edit notification comments on pull requests, go to your Netlify site settings.

Copy link
Collaborator

@ppbaena ppbaena left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Thanks for the deep revision @antgamdia!

@antgamdia
Copy link
Contributor Author

antgamdia commented Jan 9, 2023

Thanks! Merging as it is solely a change in the website and the failing test is just the Flux one (see #5827).

Done :)

image

@antgamdia antgamdia merged commit 12914fc into vmware-tanzu:main Jan 9, 2023
@antgamdia antgamdia deleted the md-lint-fixes branch January 9, 2023 16:41
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants