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

Can't use extensions from CLI #2617

Closed
abdullahkhalids opened this issue Oct 24, 2022 · 4 comments · Fixed by #2629 or #2937
Closed

Can't use extensions from CLI #2617

abdullahkhalids opened this issue Oct 24, 2022 · 4 comments · Fixed by #2629 or #2937

Comments

@abdullahkhalids
Copy link

Describe the feature
I would like the ability to use extensions from the CLI version of the marked. It might already be possible, but there is no documentation to this effect in the CLI help.

Why is this feature necessary?
Extensions can be used in the other ways of calling marked. They should also be usable from the CLI.

Describe alternatives you've considered
N/A

@UziTech
Copy link
Member

UziTech commented Oct 24, 2022

There is no current way to use extensions in the cli. I don't know if it is a good idea to allow everything that can be done programmatically in the cli. It seems like it would be easier to create a script that does it programmatically than use the cli at that point.

@abdullahkhalids
Copy link
Author

My particular use case is that I am developing on python. While, I could conceivably create a boiler plate script to run node and then call it from python, it would be much easier to just call the marked cli in a line and be done with it.

But I understand if this is low priority. Thanks in any case.

@UziTech
Copy link
Member

UziTech commented Oct 25, 2022

Considering that you would have to write the extensions in nodejs anyway I don't think it would be much harder to just do it programmatically. I think it would be too much to maintain to send npm packages to the cli and have it try to download and import them correctly in every situation.

@UziTech UziTech linked a pull request Oct 29, 2022 that will close this issue
@abdullahkhalids
Copy link
Author

Thanks

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

Successfully merging a pull request may close this issue.

2 participants