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

Support Markdown review format #188

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

Conversation

dirtystylus
Copy link

Hi—I’m working on a project (https://www.markllobrera.com/posts/letterboxd-to-markdown/) and I wanted the review to be in Markdown format, so I took a stab at implementing that. It uses a reviewFormat parameter and if that is set to markdown I call a new method getReviewAsMarkdown(). There’s probably a more elegant way to do this, but happy to discuss my use case if you have questions!

@dirtystylus dirtystylus marked this pull request as draft March 27, 2024 12:45
@dirtystylus dirtystylus marked this pull request as ready for review March 27, 2024 12:45
@zoetrope69
Copy link
Owner

Great stuff, I like the usecase of Eleventy

Do you think we could add a basic test for this and add the new parameter to the documentation. An example of the output in the README would be good?

@dirtystylus
Copy link
Author

@zoetrope69 Hi! Clarifying: were you asking for my assistance for either or both of those items (I think both are worthwhile)?

I can help with the README part. I’m not as familiar with writing tests for node so would defer to you for that aspect (happy to learn, though)

@zoetrope69
Copy link
Owner

sure, i can write the tests if you update the readme

@zoetrope69
Copy link
Owner

thank you!

@dirtystylus
Copy link
Author

@zoetrope69 Updated the README!

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