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

Add config argument to specify valid markdown flavors #1424

Merged
merged 3 commits into from
Feb 21, 2025

Conversation

n8layman
Copy link
Contributor

This PR attempts to address Problem 2 raised by @MilesMcBain in #1399 by implementing a configuration option, e.g., "foam.supportedLanguages": ["markdown", "quarto", "rmarkdown"],, to specify valid markdown-like flavors as suggested in #924. This is my first PR to an open-source project, so please review carefully and let me know if there are any additional steps, tests, or changes needed!

@n8layman n8layman changed the title Add Add config argument to specify valid markdown flavors Feb 12, 2025
Copy link
Collaborator

@riccardoferretti riccardoferretti left a comment

Choose a reason for hiding this comment

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

Looking good! Thanks for contributing

@riccardoferretti
Copy link
Collaborator

Can you please fix the typo reporting? (see foam-vscode/package.json, typo.toml, and https://github.com/crate-ci/typos/blob/master/docs/reference.md )

@n8layman
Copy link
Contributor Author

Sure! I used yarn to package the extension and was getting a warning about two lock files in a single project. I just renamed the npm one to disable it temporarily. Is it alright to remove the npm lock file entirely?

Copy link
Contributor Author

@n8layman n8layman left a comment

Choose a reason for hiding this comment

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

Okay it looks like removing that file takes care of the typo test.

@n8layman
Copy link
Contributor Author

Is there anything else needed?

@riccardoferretti riccardoferretti merged commit dd467ce into foambubble:master Feb 21, 2025
3 checks passed
@riccardoferretti
Copy link
Collaborator

@allcontributors add @n8layman for code

Copy link
Contributor

@riccardoferretti

I've put up a pull request to add @n8layman! 🎉

@riccardoferretti
Copy link
Collaborator

all good, thanks @n8layman !

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