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

Consider archiving this repository #106

Open
chrisjsewell opened this issue Jun 23, 2021 · 5 comments
Open

Consider archiving this repository #106

chrisjsewell opened this issue Jun 23, 2021 · 5 comments

Comments

@chrisjsewell
Copy link
Member

Similar to executablebooks/rst2myst#5,
I would consider archiving this repository, in favour of https://github.com/executablebooks/rst-to-myst

although I note this one has been updated more recently, so won't do it without discussion

cc @choldgraf @AakashGfude @mmcky @jstac for comment

@welcome
Copy link

welcome bot commented Jun 23, 2021

Thanks for opening your first issue here! Engagement like this is essential for open source projects! 🤗

If you haven't done so already, check out EBP's Code of Conduct. Also, please try to follow the issue template as it helps other community members to contribute more effectively.

If your issue is a feature request, others may react to it, to raise its prominence (see Feature Voting).

Welcome to the EBP community! 🎉

@choldgraf
Copy link
Member

I don't understand the comparative differences between this and rst-to-myst, so I'll defer to others there. But in general, if they have heavily overlapping functionality, I am +1 on deprecating one of them. One possibility is that if this repository is still needed by the QuantEcon team, we could just move it to the QE org?

@mmcky
Copy link
Member

mmcky commented Jun 23, 2021

@chrisjsewell my plan is to move this to QuantEcon org once we are finished migrating all our major projects.

sphinx-tomyst has been a very useful migration tool.

@choldgraf this extension basically allowed you to add sphinx_tomyst as a sphinx extension and run make myst to generate a myst version of a sphinx project. But I haven't used rst-to-myst much to see how it migrates projects. I think direct translation of rst-to-myst is technically superior approach so happy to move sphinx-tomyst to the QuantEcon organisation. The downside of sphinx-tomyst is it mainly works for core rst (i.e. docutils + sphinx) as it infers from sphinx.ast

I was thinking of writing a migration guide for sphinx rst projects so I will certainly give rst-to-myst a spin. @chrisjsewell does this work well for migrating projects in bulk? Last time I gave it a spin it translated single files.

@chrisjsewell
Copy link
Member Author

Yeh once executablebooks/rst-to-myst#18 is merged (hopefully tonight) it will essentially be a lossless conversion and also output nice formatted Markdown with https://github.com/executablebooks/mdformat-myst.
This is something you just cannot do with the "standard" docutils AST, which this package works on, because there is literally no way to know for sure what the original text was e.g. what the original roles/directives used were.
Obviously its great that it worked for you but yeh, advertising it as a general tool, you would very rapidly rack up people opening bug issues

does this work well for migrating projects in bulk? Last time I gave it a spin it translated single files.

still single file at present, but its pretty easy to add in-place conversions + file globbing, and so have a command that walks through and converts every file in a project, i.e. there is no requirement for any extra "inter-document" logic, you just point it at whatever text you want and it will convert it.
I'll probably also do this tonight/today as well.

@mmcky
Copy link
Member

mmcky commented Jun 24, 2021

@chrisjsewell i agree. I'm happy not to maintain this as soon as rst-to-myst is good to go. Happy to also debug and test any bulk conversion / project conversion feature. Thanks @chrisjsewell

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

No branches or pull requests

3 participants