swcarpentry / python-novice-gapminder

Plotting and Programming in Python
http://swcarpentry.github.io/python-novice-gapminder/
Other
162 stars 428 forks source link

fix: remove leftover Jekyll `auto_ids` directive #659

Closed alee closed 10 months ago

alee commented 10 months ago

closes #658

github-actions[bot] commented 10 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/swcarpentry/python-novice-gapminder/compare/md-outputs..md-outputs-PR-659

The following changes were observed in the rendered markdown documents:

 md5sum.txt   | 2 +-
 reference.md | 1 -
 2 files changed, 1 insertion(+), 2 deletions(-)
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 2023-08-18 22:27:52 +0000

zkamvar commented 10 months ago

Hi @alee, I noticed that the commit is coming from @carpentries-bot and I wonder if you might accidentally be experiencing this issue: https://github.com/carpentries/sandpaper/issues/449

You might want to check that you have sandpaper >= 0.12.3 and then check and remove the user and email from this repository's .git/config

alee commented 10 months ago

good catch! I was mucking around with a containerized version of this some time ago and it must have modified .git/config. I'll close and resubmit