Skip to content

Support graymatter data in asciidoc files #33403

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

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

Conversation

halkeye
Copy link
Contributor

@halkeye halkeye commented Oct 2, 2021

Description

Add support for processing graymatter headers inside of asciidoc files

Documentation

At jenkins we use asciidoc for the main website. its already using frontmatter content all over the place. I want to convert to gatsby, so i need to support this

Example: https://github.com/jenkins-infra/jenkins.io/blob/master/content/blog/2021/09/2021-09-04-wiki-attacked.adoc

[X] Update any references, if relevant. This includes Guides and Gatsby Internals docs.
[X] Tag @gatsbyjs/documentation for review, pairing, polishing of the documentation

Related Issues

@gatsbot gatsbot bot added the status: triage needed Issue or pull request that need to be triaged and assigned to a reviewer label Oct 2, 2021
@halkeye halkeye force-pushed the asciigraymatter branch 2 times, most recently from 31eb12d to 5fb9924 Compare October 2, 2021 01:24
@halkeye
Copy link
Contributor Author

halkeye commented Oct 2, 2021

I don't think the failed tests are related to my change :(

@wardpeet
Copy link
Contributor

wardpeet commented Oct 4, 2021

Thank you! Is it possible to add an example of grasmatten to the using-asciidoc example 🙏 This allows me to test the PR

@halkeye
Copy link
Contributor Author

halkeye commented Oct 4, 2021

Sure. I can take a stab tomorrow night. Can you point me where the examples are?

@@ -98,6 +113,9 @@ export const pageQuery = graphql`
authorInitials
email
}
frontmatter {
source
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

none of the example asciidoctor files have frontmatter yet right?

Copy link
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

oh shoot i forgot to git add, which explained why ci failed

@halkeye
Copy link
Contributor Author

halkeye commented Oct 5, 2021

Okay, missing files re-added, then rebased ontop of master again

@TylerBarnes TylerBarnes removed the status: triage needed Issue or pull request that need to be triaged and assigned to a reviewer label Aug 19, 2022
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.

4 participants