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

fix(docz-core): properly support node_modules for multiple file patterns #1315

Merged
merged 2 commits into from
Dec 13, 2019

Conversation

axe312ger
Copy link
Contributor

@axe312ger axe312ger commented Dec 12, 2019

Description

Currently docs from node_modules are only supported when the src contains node_modules

This changes the code so that either src or any pattern can contain docs from node_modules

My config:

{
  resolve: `gatsby-theme-docz`,
  options: {
    files: [
      `src/components/mdx/**/docs/*.{md,markdown,mdx}`,
      `node_modules/@gatsby-mdx-suite/*/docs/*.{md,markdown,mdx}`,
      `../../node_modules/@gatsby-mdx-suite/*/docs/*.{md,markdown,mdx}`,
    ]
  },
}

@rakannimer
Copy link
Contributor

Awesome ! I added a src check to avoid breaking any setups that relied on it.

@rakannimer rakannimer merged commit 4d9f9b3 into doczjs:master Dec 13, 2019
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants