carpentries-lab / good-enough-practices

Good Enough Practices in Scientific Computing
https://carpentries-lab.github.io/good-enough-practices/
Other
17 stars 20 forks source link

DOI to config.yaml #146

Closed ewallace closed 6 months ago

ewallace commented 8 months ago

Add doi for lesson to config.yaml doi: 10.5281/zenodo.10783026

https://zenodo.org/doi/10.5281/zenodo.10783025

fixes #126

github-actions[bot] commented 8 months ago

Thank you!

Thank you for your pull request :smiley:

:robot: This automated message can help you check the rendered files in your submission for clarity. If you have any questions, please feel free to open an issue in {sandpaper}.

If you have files that automatically render output (e.g. R Markdown), then you should check for the following:

Rendered Changes

:mag: Inspect the changes: https://github.com/carpentries-lab/good-enough-practices/compare/md-outputs..md-outputs-PR-146

The following changes were observed in the rendered markdown documents:

 config.yaml | 4 ++++
 md5sum.txt  | 2 +-
 2 files changed, 5 insertions(+), 1 deletion(-)
What does this mean? If you have source files that require output and figures to be generated (e.g. R Markdown), then it is important to make sure the generated figures and output are reproducible. This output provides a way for you to inspect the output in a diff-friendly manner so that it's easy to see the changes that occur due to new software versions or randomisation.

:stopwatch: Updated at 2024-03-07 15:42:29 +0000

tobyhodges commented 8 months ago

Looks good to me

Screenshot 2024-03-06 at 10 26 46

Let me look into why that build test is failing, because everything ran mine on my local system...

tobyhodges commented 8 months ago

Confirming that the build error is unrelated to the changes in this PR. It should be fixed in the next 24 hours, and I believe this PR is safe to merge.