Closed maneesha closed 8 months ago
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:
:mag: Inspect the changes: https://github.com/LibraryCarpentry/lc-open-refine/compare/md-outputs..md-outputs-PR-327
The following changes were observed in the rendered markdown documents:
11-using-arrays-transformations.md | 3 +--
md5sum.txt | 2 +-
2 files changed, 2 insertions(+), 3 deletions(-)
:stopwatch: Updated at 2023-12-15 15:32:30 +0000
It looks like @maneesha was using the All in One view (which I use as an instructor, too). Though it's actually the part in episode 12, on exporting data. Is that correct?
And @ostephens is verifying the link should go towards: https://librarycarpentry.org/lc-open-refine/03-working-with-data.html#choosing-a-good-separator
Also, there's an extra full stop at the end of that sentence.
Will editing the aio file also update the right episode file?
As an internal reference, does this need the .md instead of .html? I don't recall how that works with hashtags to subsections.
Though it's actually the part in episode 12, on exporting data. Is that correct?
Just for clarity it's in episode 11 (which you link to)
Will editing the aio file also update the right episode file?
It would not - but the edit has been done on the correct file episodes/11-using-arrays-transformations.md
so it is being done in the right place as far as I can see
As an internal reference, does this need the .md instead of .html? I don't recall how that works with hashtags to subsections.
Good catch - yes this should use the internal link syntax and that will work with the hash / fragment as well. See https://carpentries.github.io/sandpaper-docs/episodes.html#internal-links
Broken link to "Working with Data" from https://librarycarpentry.org/lc-open-refine/aio.html#arrays