-
Notifications
You must be signed in to change notification settings - Fork 90
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
GitHub actions to work with sphinx docs and conda packages #135
Comments
Thanks for the suggestion and glad you are finding the cookiecutter helpful. Im reluctant to add these two items to the cookiecutter directly for a couple reasons:
I do like both options personally and would find uses for them in various projects, but I don't know if we should add them to this explicit cookiecutter as "recommended" items. While true, we do have some default Sphinx files, we don't actually do anything for building and publishing because there can be a fair amount of nuance with Sphinx, especially as versions change and potentially break plugins (a nightmare I have had to chase down a few times in some projects) Conda building is something I don't think is straightforward enough to give a broad recommendation for conda-build on its own. Our README even explicitly advises against With all that said, it may be worth adding documentation and options in the central or nested READMEs to make suggestions on how people can interface with these tools. I'm also open to counter arguments, especially on the Conda side of things, its just going to take a very strong case for me to be comfortable with pointing people directly at |
Thanks for your clear and kind answer. I understand and share your opinions. Thanks again for your feedback. We really appreciate it, as we strongly appreciate the MolSSI efforts to show us the best practices to develop and maintain our tools. We are learning a lot little by little with your materials. |
@Lnaden I just wanted to revive this issue, since I'm 100% in agreement with @dprada that this would be super useful. It's so easy to publish with GitHub Pages that automating docs builds with GitHub Actions is the "killer app" in making it truly easy for users to just edit a couple of files and have useful docs (including API docs!) appear online. Making it easy for users to add and maintain useful documentation isn't scope creep here---it's an essential part of software development. If you software doesn't have documentation, it means you don't want or care about other people using it. We want to incentivize users to do the right thing, and the best way to do that is to make the barrier to doing so near zero. It doesn't necessarily have to be sphinx---#192 also suggests MkDocs as a simpler alternative. But having at least one of these mechanisms automatically build and publish docs via GitHub Actions seems like exactly what is needed to make sure every project has dead-simple high quality online docs available. |
Will the search still work if it is on github pages? readthedocs does some fancy indexing to make the search fast and useful https://docs.readthedocs.io/en/stable/development/search.html they do have all the code open source and there is a self-hosting option for Elasticsearch if you don't want to pay for the SaaS |
Hi! Our lab is recently working with your cookie-cutter to begin every new project. Thanks for the initiative and the work to keep it operative and well documented.
We'd like to suggest the inclusion of two GitHub actions that might be useful for most of the users. We missed the automatization of the two following workflows:
Because of this reason, we developed, based on other public actions, two actions to cover these needs:
https://github.com/uibcdf/action-sphinx-docs-to-gh-pages
https://github.com/uibcdf/action-build-and-upload-conda-packages
I am not sure if every user of this cookie-cutter would found these useful. Maybe not, what's your opinion?
If you think that these workflows can be helpful to other colleagues. Please, feel free to copy or re-cook the scripts to be included here. No problem. Maybe MolSSI could have their own GitHub actions.
Thanks again for offering this tool to the community.
The text was updated successfully, but these errors were encountered: