Skip to content

[docs] Enforce correct git diff format#34765

Merged
oliviertassinari merged 1 commit into
mui:masterfrom
oliviertassinari:git-diff-format
Oct 21, 2022
Merged

[docs] Enforce correct git diff format#34765
oliviertassinari merged 1 commit into
mui:masterfrom
oliviertassinari:git-diff-format

Conversation

@oliviertassinari

@oliviertassinari oliviertassinari commented Oct 15, 2022

Copy link
Copy Markdown
Member

Enforce the git diff format with a custom rule for https://github.com/DavidAnson/markdownlint. It's simply about respecting git show output. The new rule is in packages/markdownlint-rule-mui/git-diff.js. I felt that I was frequently leaving comments about this aspect, this will save me time 😁.

The discussion in mui/mui-x#6523 (comment) covers a bit more the value of it. It's a detail, but we are a UI company, people look up to us to care about these small details.

@oliviertassinari oliviertassinari added the docs Improvements or additions to the documentation. label Oct 15, 2022
@mui-bot

mui-bot commented Oct 15, 2022

Copy link
Copy Markdown

@oliviertassinari
oliviertassinari marked this pull request as ready for review October 15, 2022 11:30
@oliviertassinari oliviertassinari changed the title [docs] Fix git diff formats [docs] Enforce correct git diff formats Oct 15, 2022
@oliviertassinari oliviertassinari changed the title [docs] Enforce correct git diff formats [docs] Enforce correct git diff format Oct 15, 2022
@oliviertassinari
oliviertassinari enabled auto-merge (squash) October 21, 2022 15:01
@oliviertassinari
oliviertassinari merged commit 02451c6 into mui:master Oct 21, 2022
@oliviertassinari
oliviertassinari deleted the git-diff-format branch October 21, 2022 15:02
daniel-rabe pushed a commit to daniel-rabe/material-ui that referenced this pull request Nov 29, 2022
feliperli pushed a commit to jesrodri/material-ui that referenced this pull request Dec 6, 2022
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

docs Improvements or additions to the documentation.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants