Skip to content

Add mdx support to cli #411

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

Open
wants to merge 2 commits into
base: master
Choose a base branch
from
Open

Add mdx support to cli #411

Changes from all commits
Commits
Show all changes
2 commits
Select commit Hold shift + click to select a range
f25ff74
add mdx support to cli
TudorCRL Feb 18, 2025
2a6a5a3
recursive mdx check
TudorCRL Feb 19, 2025
File filter

Filter by extension

Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
11 changes: 6 additions & 5 deletions markdown-link-check
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -96,13 +96,13 @@ function commaSeparatedCodesList(value, dummyPrevious) {
/**
* Load all files in the rootFolder and all subfolders that end with .md
*/
function loadAllMarkdownFiles(rootFolder = '.') {
function loadAllMarkdownFiles(rootFolder = '.', includeMdx = false) {
const files = [];
fs.readdirSync(rootFolder).forEach(file => {
const fullPath = path.join(rootFolder, file);
if (fs.lstatSync(fullPath).isDirectory()) {
files.push(...loadAllMarkdownFiles(fullPath));
} else if (fullPath.endsWith('.md')) {
files.push(...loadAllMarkdownFiles(fullPath, includeMdx));
} else if (fullPath.endsWith('.md') || (includeMdx && fullPath.endsWith('mdx'))) {
files.push(fullPath);
}
});
Expand All @@ -127,6 +127,7 @@ function getInputs() {
.option('-r, --retry', 'retry after the duration indicated in \'retry-after\' header when HTTP code is 429')
.option('--reporters <names>', 'specify reporters to use', commaSeparatedReportersList)
.option('--projectBaseUrl <url>', 'the URL to use for {{BASEURL}} replacement')
.option('--include-mdx', 'check links in mdx files as well as md files')
.arguments('[filenamesOrDirectorynamesOrUrls...]')
.action(function (filenamesOrUrls) {
let filenameForOutput;
Expand All @@ -148,7 +149,7 @@ function getInputs() {
}
}

const { ignore } = program.opts();
const { ignore, includeMdx } = program.opts();

for (const filenameOrUrl of filenamesOrUrls) {
filenameForOutput = filenameOrUrl;
Expand Down Expand Up @@ -177,7 +178,7 @@ function getInputs() {
let files = [];

if (fs.statSync(filenameOrUrl).isDirectory()){
files = loadAllMarkdownFiles(filenameOrUrl)
files = loadAllMarkdownFiles(filenameOrUrl, includeMdx)
} else {
files = [filenameOrUrl]
}
Expand Down