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

change tutorials to link to cookbook webpage #96

Merged
merged 2 commits into from
Apr 25, 2023

Conversation

sef43
Copy link
Contributor

@sef43 sef43 commented Apr 21, 2023

as discussed in openmm/openmm-cookbook#14 this changes the tutorials section to link to the tutorials on the cookbook webpage, and adds link to the cookbook and tutorials in the documentation section.

deployment can be seen here: https://sef43.github.io/openmm-org/

@peastman
Copy link
Member

I think we should just remove the Tutorials tab. It no longer serves any purpose.

@sef43
Copy link
Contributor Author

sef43 commented Apr 24, 2023

I have removed the tutorials tab

@peastman
Copy link
Member

I see you added the cookbook and tutorials to the development docs section. What's the workflow for managing them? They aren't tied to a release in the same way the rest of the documentation is. When do the development versions get built, and when do they get promoted to latest?

@sef43
Copy link
Contributor Author

sef43 commented Apr 24, 2023

Yes that is true, the cookbook has its own release cycle, the main branch of the cookbook is automatically deployed to githubpages as the cookbook/dev/ . It is promoted to latest when a release is done on the cookbook github repo.

We can remove the links to the dev versions from here and just leave the latest versions if you think that is better?

@peastman
Copy link
Member

Either way. I just wanted to understand what the intended workflow is. If you want to include the links, that's fine.

@peastman peastman merged commit d747c8b into openmm:master Apr 25, 2023
@sef43 sef43 deleted the tutorial_links branch April 27, 2023 10:46
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

Successfully merging this pull request may close these issues.

2 participants