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

Different open and close delimiters #18

Conversation

Jawell
Copy link
Contributor

@Jawell Jawell commented May 12, 2024

The JS library allows the use of different delimiters for the start and the end: https://www.npmjs.com/package/gray-matter#optionsdelimiters

What do you think? Could we add the same functionality?

Copy link
Collaborator

@yuchanns yuchanns left a comment

Choose a reason for hiding this comment

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

Thank you!

src/matter.rs Outdated Show resolved Hide resolved
@yuchanns
Copy link
Collaborator

LGTM.

Just one more step left. Could you please sign all your commits with the GPG key?

FYI: https://docs.github.com/en/authentication/managing-commit-signature-verification/signing-commits

@Jawell
Copy link
Contributor Author

Jawell commented May 16, 2024

LGTM.

Just one more step left. Could you please sign all your commits with the GPG key?

FYI: https://docs.github.com/en/authentication/managing-commit-signature-verification/signing-commits

I didn't notice that I have to do it 🫠
So, yes, I'll update the PR

@Jawell Jawell force-pushed the different-start-and-end-delimiters branch from 55952dd to 6af54d7 Compare May 17, 2024 06:09
@yuchanns yuchanns merged commit 63b1b78 into the-alchemists-of-arland:main May 17, 2024
9 checks passed
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