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

tools: notify user if format-md needs to be run #40647

Merged
merged 0 commits into from
Nov 2, 2021

Conversation

Trott
Copy link
Member

@Trott Trott commented Oct 29, 2021

The first commit here is #40645 which will need to land first (or can be closed in favor of this PR).

Make lint-md fail and tell the user to run format-md if they have misformatted markdown files.

@Trott Trott added the blocked PRs that are blocked by other issues or PRs. label Oct 29, 2021
@nodejs-github-bot nodejs-github-bot added doc Issues and PRs related to the documentations. tools Issues and PRs related to the tools directory. labels Oct 29, 2021
@Trott
Copy link
Member Author

Trott commented Oct 29, 2021

Blocked on #40645 or at least blocked on the same things that that PR is blocked on.

@Trott Trott added author ready PRs that have at least one approval, no pending requests for changes, and a CI started. and removed blocked PRs that are blocked by other issues or PRs. labels Oct 31, 2021
@Trott Trott force-pushed the enforce-format-md branch 2 times, most recently from 69808ca to 763c3d5 Compare November 1, 2021 22:44
Trott added a commit that referenced this pull request Nov 2, 2021
PR-URL: #40647
Reviewed-By: Antoine du Hamel <[email protected]>
Reviewed-By: Michaël Zasso <[email protected]>
Trott added a commit that referenced this pull request Nov 2, 2021
This will help enforce formatting of markdown files.

PR-URL: #40647
Reviewed-By: Antoine du Hamel <[email protected]>
Reviewed-By: Michaël Zasso <[email protected]>
@Trott Trott closed this Nov 2, 2021
@Trott Trott merged commit d4aca08 into nodejs:master Nov 2, 2021
@Trott
Copy link
Member Author

Trott commented Nov 2, 2021

Landed in 6d0b4df...d4aca08

@Trott Trott deleted the enforce-format-md branch November 2, 2021 02:05
targos pushed a commit that referenced this pull request Nov 6, 2021
PR-URL: #40647
Reviewed-By: Antoine du Hamel <[email protected]>
Reviewed-By: Michaël Zasso <[email protected]>
targos pushed a commit that referenced this pull request Nov 6, 2021
This will help enforce formatting of markdown files.

PR-URL: #40647
Reviewed-By: Antoine du Hamel <[email protected]>
Reviewed-By: Michaël Zasso <[email protected]>
targos pushed a commit that referenced this pull request Nov 6, 2021
If there is not a blank line before it, the comment does not re-enable
the rule.

PR-URL: #40647
Reviewed-By: Antoine du Hamel <[email protected]>
Reviewed-By: Michaël Zasso <[email protected]>
@targos targos mentioned this pull request Nov 8, 2021
danielleadams pushed a commit that referenced this pull request Jan 29, 2022
PR-URL: #40647
Reviewed-By: Antoine du Hamel <[email protected]>
Reviewed-By: Michaël Zasso <[email protected]>
danielleadams pushed a commit that referenced this pull request Jan 30, 2022
This will help enforce formatting of markdown files.

PR-URL: #40647
Reviewed-By: Antoine du Hamel <[email protected]>
Reviewed-By: Michaël Zasso <[email protected]>
danielleadams pushed a commit that referenced this pull request Jan 30, 2022
If there is not a blank line before it, the comment does not re-enable
the rule.

PR-URL: #40647
Reviewed-By: Antoine du Hamel <[email protected]>
Reviewed-By: Michaël Zasso <[email protected]>
danielleadams pushed a commit that referenced this pull request Feb 1, 2022
This will help enforce formatting of markdown files.

PR-URL: #40647
Reviewed-By: Antoine du Hamel <[email protected]>
Reviewed-By: Michaël Zasso <[email protected]>
danielleadams pushed a commit that referenced this pull request Feb 1, 2022
If there is not a blank line before it, the comment does not re-enable
the rule.

PR-URL: #40647
Reviewed-By: Antoine du Hamel <[email protected]>
Reviewed-By: Michaël Zasso <[email protected]>
@danielleadams danielleadams mentioned this pull request Feb 1, 2022
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
author ready PRs that have at least one approval, no pending requests for changes, and a CI started. doc Issues and PRs related to the documentations. tools Issues and PRs related to the tools directory.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants