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.

Sign up for GitHub

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

Jump to bottom

Fix MD009: No trailing spaces #10862

Merged
merged 1 commit into from
Dec 1, 2021
Merged

Fix MD009: No trailing spaces #10862

merged 1 commit into from
Dec 1, 2021

Conversation

Elchi3
Copy link
Member

@Elchi3 Elchi3 commented Nov 29, 2021

Summary

Remove trailing spaces from markdown files, per Markdown-lint rule MD009.

Motivation

This PR is an exercise for #10787.

@Elchi3 Elchi3 requested a review from a team November 29, 2021 12:27
@Elchi3 Elchi3 requested review from a team as code owners November 29, 2021 12:27
@Elchi3 Elchi3 requested a review from a team November 29, 2021 12:27
@Elchi3 Elchi3 requested a review from a team as a code owner November 29, 2021 12:27
@Elchi3 Elchi3 requested review from schalkneethling, estelle, teoli2003, rebloor and wbamberg and removed request for a team November 29, 2021 12:27
@github-actions github-actions bot added Content:Accessibility Accessibility docs Content:CSS Cascading Style Sheets docs Content:Glossary Glossary entries Content:HTML Hypertext Markup Language docs Content:HTTP HTTP docs Content:JS JavaScript docs Content:Learn Learning area docs Content:Learn:Django Learning area Django docs Content:Learn:Express Learning area Express docs labels Nov 29, 2021
@github-actions github-actions bot added Content:MathML MathML docs Content:Other Any docs not covered by another "Content:" label Content:SVG SVG docs Content:WebAPI Web API docs Content:WebExt WebExtensions docs labels Nov 29, 2021
@github-actions

This comment has been minimized.

Sign in to view
ddbeck
ddbeck reviewed Nov 29, 2021
View reviewed changes
Copy link
Contributor

@ddbeck ddbeck left a comment

Choose a reason for hiding this comment

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

This feels pretty sustainable. I've reviewed the first 70 files (through files/en-us/web/accessibility/aria/attributes/index.md) and it took me about 10 minutes. I can come back to this later, but I thought I'd note my progress.

rebloor
rebloor approved these changes Nov 29, 2021
View reviewed changes
Copy link
Contributor

@rebloor rebloor left a comment

Choose a reason for hiding this comment

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

Checked out the changes on the web extension pages, all looks fine.

@estelle estelle merged commit 0f1f6f7 into mdn:main Dec 1, 2021
@Elchi3 Elchi3 deleted the md009 branch December 1, 2021 08:37
@rayw000 rayw000 mentioned this pull request Dec 1, 2021
Closed
3 tasks
@ddbeck ddbeck removed Content:Learn:Express Learning area Express docs Content:Learn:Django Learning area Django docs labels Mar 31, 2022
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@ddbeck ddbeck ddbeck left review comments

@estelle estelle estelle approved these changes

@rebloor rebloor rebloor approved these changes

@schalkneethling schalkneethling Awaiting requested review from schalkneethling

@teoli2003 teoli2003 Awaiting requested review from teoli2003 teoli2003 is a code owner automatically assigned from mdn/yari-content-http

@wbamberg wbamberg Awaiting requested review from wbamberg wbamberg is a code owner automatically assigned from mdn/yari-content-web-api

Assignees
No one assigned
Labels
Content:Accessibility Accessibility docs Content:CSS Cascading Style Sheets docs Content:Glossary Glossary entries Content:HTML Hypertext Markup Language docs Content:HTTP HTTP docs Content:JS JavaScript docs Content:Learn Learning area docs Content:MathML MathML docs Content:Other Any docs not covered by another "Content:" label Content:SVG SVG docs Content:WebAPI Web API docs Content:WebExt WebExtensions docs
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
4 participants
@Elchi3 @ddbeck @estelle @rebloor