unist-builder and prismjs should be dependencies?

[titaniumbones]

just trying out the package (working towards this issue for a vscode plugin: https://github.com/vscode-org-mode/vscode-org-mode/issues/133). Tests failed until I installed unist-builder and prismjs, so I'm assuming they need to be added to the dependencies list?

[xiaoxinghu]

@titaniumbones which package are you talking about? oast-to-hast is the only one who depend on unist-builder and prismjs, and they are in the dep list. orga itself doesn't have any dependency.

[titaniumbones]

When I tried to run the orgsjs test suite, most failed until I installed those two packages manually.

I will try to look at this more carefully though. I was just reporting the failure.

On Mar. 22, 2019 4:51 a.m., Xiaoxing Hu [email protected] wrote:

@titaniumboneshttps://github.com/titaniumbones which package are you talking about? oast-to-hast is the only one who depend on unist-builder and prismjs, and they are in the dep list. orga itself doesn't have any dependency.

— You are receiving this because you were mentioned. Reply to this email directly, view it on GitHubhttps://github.com/xiaoxinghu/orgajs/issues/21#issuecomment-475539419, or mute the threadhttps://github.com/notifications/unsubscribe-auth/AAWPNCADr9jeb04FmVj3AhCYwKwWi9b6ks5vZJl2gaJpZM4b78i9.

[xiaoxinghu]

I have updated the readme; please reference the Contribute part to make sure you are running the tests the right way. I think your failure is most likely due to using npm instead of yarn. I am using yarn because the workspace feature made the development of gatsby plugin easy (and possible). I'd like to switch back to npm when their equivalent of workspace feature is ready. (They said it's going to be included in the next major version.)