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

Using context for extensions #777

Open
leotsarev opened this issue Mar 5, 2024 · 4 comments
Open

Using context for extensions #777

leotsarev opened this issue Mar 5, 2024 · 4 comments

Comments

@leotsarev
Copy link
Contributor

When my extension is rendering, it needs access to MarkdownParserContext.
Now I has to pass context through Inline objects, and i don't like it.
Is there way to extension to access it?

@xoofx
Copy link
Owner

xoofx commented Mar 14, 2024

As per the name MarkdownParserContext, it was added for accessing it from parsing. It should be possible to modify the code to have this context passed around. We might have to rename it though (e.g MardownContext or something)

@leotsarev
Copy link
Contributor Author

We might have to rename it though (e.g MardownContext or something)
It would be a breaking change...

@xoofx
Copy link
Owner

xoofx commented Mar 15, 2024

It would be a breaking change...

Markdig is not 1.0, so we don't make a promise about not making breaking changes.

@xoofx
Copy link
Owner

xoofx commented Mar 15, 2024

But that doesn't mean that we can't find a solution that provide backward compatibility. If there is a path, that's better for sure. 🙂

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

No branches or pull requests

2 participants