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

Minor docs cleanup #1047

Merged
merged 4 commits into from
Nov 14, 2024
Merged

Minor docs cleanup #1047

merged 4 commits into from
Nov 14, 2024

Conversation

esoteric-ephemera
Copy link
Contributor

Correcting out of date info / incorrect links in the docs:

  • Should close #1043, wherein the VASP workflows link to makers in other code bases
  • Some out of date OpenMM info - @orionarcher can you double check my changes are right? The docs were pointing to your branch for install info

@JaGeo
Copy link
Member

JaGeo commented Nov 8, 2024

@esoteric-ephemera Thank you very much! 😀

@JaGeo
Copy link
Member

JaGeo commented Nov 12, 2024

Waiting for @orionarcher to confirm before merging.

@orionarcher
Copy link
Contributor

LGTM

Copy link
Member

@janosh janosh left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

thanks @esoteric-ephemera! 👍

@janosh janosh enabled auto-merge (squash) November 14, 2024 21:04
@janosh janosh added the docs Improvements or additions to documentation label Nov 14, 2024
@janosh janosh merged commit 67efb7f into materialsproject:main Nov 14, 2024
15 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
docs Improvements or additions to documentation
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Documentation Inconsistency in ElasticMaker Input Set Modification Example.
4 participants