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

XS✔ - Fix markdownlint action #7370

Merged
merged 1 commit into from
Nov 30, 2023
Merged

XS✔ - Fix markdownlint action #7370

merged 1 commit into from
Nov 30, 2023

Conversation

JackDevAU
Copy link
Member

  1. What triggered this change? (PBI link, Email Subject, conversation + reason, etc)

errors with multiple files and using the \n char

  1. What was changed?

replace \n with space

  1. Did you do pair or mob programming (and list names)?

@github-actions github-actions bot added the Age: 🥚 - New About 2 hours old label Nov 29, 2023
Copy link
Contributor

Hi @JackDevAU,

There were some Markdown Linting Errors in your pull request:

File Error Rule
rules/8-steps-to-scrum/rule.md Line 22 MD036/no-emphasis-as-heading/no-emphasis-as-header Emphasis used instead of a heading [Context: used instead of a heading Rule MD036

Please review the errors and update your Markdown files accordingly.

Copy link
Contributor

github-actions bot commented Nov 29, 2023

PR Metrics

Thanks for keeping your pull request small.
Thanks for adding tests.

Lines
Product Code 0
Test Code 0
Subtotal 0
Ignored Code 1
Total 1

Metrics computed by PR Metrics. Add it to your Azure DevOps and GitHub PRs!

@github-actions github-actions bot changed the title [⚠️TESTING DON'T MERGE] - markdownlint action XS✔ ◾ [⚠️TESTING DON'T MERGE] - markdownlint action Nov 29, 2023
Copy link
Contributor

Hi @JackDevAU,

There were some Markdown Linting Errors in your pull request:

File Error Rule
rules/3-steps-to-a-pbi/rule.md Line 67 MD025/single-title/single-h1 Multiple top-level headings in the same document [Context: top-level headings in the same document Rule MD025
rules/3-steps-to-a-pbi/rule.md Line 69 MD025/single-title/single-h1 Multiple top-level headings in the same document [Context: top-level headings in the same document Rule MD025
rules/8-steps-to-scrum/rule.md Line 22 MD036/no-emphasis-as-heading/no-emphasis-as-header Emphasis used instead of a heading [Context: used instead of a heading Rule MD036
rules/8-steps-to-scrum/rule.md Line 78 MD025/single-title/single-h1 Multiple top-level headings in the same document [Context: top-level headings in the same document Rule MD025
rules/8-steps-to-scrum/rule.md Line 80 MD025/single-title/single-h1 Multiple top-level headings in the same document [Context: top-level headings in the same document Rule MD025

Please review the errors and update your Markdown files accordingly.

1 similar comment
Copy link
Contributor

Hi @JackDevAU,

There were some Markdown Linting Errors in your pull request:

File Error Rule
rules/3-steps-to-a-pbi/rule.md Line 67 MD025/single-title/single-h1 Multiple top-level headings in the same document [Context: top-level headings in the same document Rule MD025
rules/3-steps-to-a-pbi/rule.md Line 69 MD025/single-title/single-h1 Multiple top-level headings in the same document [Context: top-level headings in the same document Rule MD025
rules/8-steps-to-scrum/rule.md Line 22 MD036/no-emphasis-as-heading/no-emphasis-as-header Emphasis used instead of a heading [Context: used instead of a heading Rule MD036
rules/8-steps-to-scrum/rule.md Line 78 MD025/single-title/single-h1 Multiple top-level headings in the same document [Context: top-level headings in the same document Rule MD025
rules/8-steps-to-scrum/rule.md Line 80 MD025/single-title/single-h1 Multiple top-level headings in the same document [Context: top-level headings in the same document Rule MD025

Please review the errors and update your Markdown files accordingly.

@JackDevAU JackDevAU changed the title XS✔ ◾ [⚠️TESTING DON'T MERGE] - markdownlint action XS✔ - Fix markdownlint action Nov 30, 2023
@bradystroud bradystroud merged commit debff34 into main Nov 30, 2023
4 checks passed
@bradystroud bradystroud deleted the fix-markdownlint-action branch November 30, 2023 00:11
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Age: 🥚 - New About 2 hours old
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants