carpentries / instructor-training

Instructor Training
https://carpentries.github.io/instructor-training/
Other
175 stars 290 forks source link

remove earth comment.md #1557

Closed minyoungJYi closed 1 year ago

minyoungJYi commented 1 year ago

The comment about the age of the earth is unintentionally disparaging of fundamentalist Christians. Not sure it is essential to the point.

github-actions[bot] commented 1 year 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/instructor-training/compare/md-outputs..md-outputs-PR-1557

The following changes were observed in the rendered markdown documents:

 02-practice-learning.md | 4 ++--
 md5sum.txt              | 2 +-
 2 files changed, 3 insertions(+), 3 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-06-12 13:04:25 +0000

ndporter commented 1 year ago

Thanks for the submission, @minyoungJYi . This change needs some further discussion within our community before implementing.

ndporter commented 1 year ago

After discussion by both the trainers community and trainer leadership, the decision has been made to leave the example, but to provide some additional clarifications about this kind of belief. Details are in #1578. We're very grateful to @minyoungJYi for bringing this to our attention. This is moving from a PR to an issue in order to facilitate more discussion before proposing exact changes, consistent with the purpose of Git issues.

All are encouraged to contribute to issue #1578 if they wish to be part of the discussion of how to implement.