Skip to content

Update text in 1.3 and multiple suggested revisions#66

Merged
RabeaMue merged 7 commits intoLibraryCarpentry:mainfrom
morskyjezek:main
Feb 6, 2026
Merged

Update text in 1.3 and multiple suggested revisions#66
RabeaMue merged 7 commits intoLibraryCarpentry:mainfrom
morskyjezek:main

Conversation

@morskyjezek
Copy link
Contributor

@morskyjezek morskyjezek commented Nov 6, 2025

Please briefly summarise the changes made in the pull request, and the reason(s) for making these changes.

There are multiple suggestions here, but each is in its own commit, so if you all would prefer to make more incremental changes, please feel free to modify or reject.

Here's an overview of the suggestions:

If any relevant discussions have taken place elsewhere, please provide links to these.

I haven't been a contributor to this lesson previously, but I hope you all don't mind these suggestions. I've worked mostly on the LC tidy data, shell, and some other small changes previously. I've also been a maintainer on the tidy data lesson.

These changes are mostly in response to some of the issues and requests noted as possible ways to refine and move the lesson along in the stages, as discussed in PR #61

@github-actions
Copy link

github-actions bot commented Nov 6, 2025

🆗 Pre-flight checks passed 😃

This pull request has been checked and contains no modified workflow files or spoofing.

It should be safe to Approve and Run the workflows that need maintainer approval.

@morskyjezek
Copy link
Contributor Author

Hi all, just wanted to see if this was under consideration. I am working on a wikidata session for another project and was considering making some more changes. But I don't want to step on any toes. Please let me know if additional small changes that appear to address some unfinished elements would be welcome, or not. Thanks!

@RabeaMue
Copy link
Contributor

RabeaMue commented Feb 6, 2026

Hi @morskyjezek ,

thanks a lot for this PR and for taking the time to work through several of the unfinished parts of the lesson. The changes are thoughtful and directly address points that were raised earlier in the discussion around lesson maturity, especially the added instructional text in 1.3 and the work on key points and clarity.

I’m happy to merge this PR. These kinds of incremental improvements are very welcome and align well with my plan to actively continue developing the lesson toward beta.

Please feel free to open additional PRs for further small or self contained improvements. If larger structural changes come up, it would be great to flag them early in an issue so we can coordinate scope.

Thanks again for the contribution.

Best
Rabea

@RabeaMue RabeaMue merged commit 9663f6d into LibraryCarpentry:main Feb 6, 2026
1 check passed
github-actions bot pushed a commit that referenced this pull request Feb 6, 2026
Auto-generated via `{sandpaper}`
Source  : 9663f6d
Branch  : main
Author  : Rabea Müller <42644523+RabeaMue@users.noreply.github.com>
Time    : 2026-02-06 17:38:09 +0000
Message : Merge pull request #66 from morskyjezek/main

Update text in 1.3 and multiple suggested revisions
github-actions bot pushed a commit that referenced this pull request Feb 6, 2026
Auto-generated via `{sandpaper}`
Source  : 09f24ec
Branch  : md-outputs
Author  : GitHub Actions <actions@github.com>
Time    : 2026-02-06 17:38:53 +0000
Message : markdown source builds

Auto-generated via `{sandpaper}`
Source  : 9663f6d
Branch  : main
Author  : Rabea Müller <42644523+RabeaMue@users.noreply.github.com>
Time    : 2026-02-06 17:38:09 +0000
Message : Merge pull request #66 from morskyjezek/main

Update text in 1.3 and multiple suggested revisions
@morskyjezek
Copy link
Contributor Author

Thanks, this is a great lesson!

github-actions bot pushed a commit that referenced this pull request Feb 10, 2026
Auto-generated via `{sandpaper}`
Source  : 09f24ec
Branch  : md-outputs
Author  : GitHub Actions <actions@github.com>
Time    : 2026-02-06 17:38:53 +0000
Message : markdown source builds

Auto-generated via `{sandpaper}`
Source  : 9663f6d
Branch  : main
Author  : Rabea Müller <42644523+RabeaMue@users.noreply.github.com>
Time    : 2026-02-06 17:38:09 +0000
Message : Merge pull request #66 from morskyjezek/main

Update text in 1.3 and multiple suggested revisions
github-actions bot pushed a commit that referenced this pull request Feb 17, 2026
Auto-generated via `{sandpaper}`
Source  : 09f24ec
Branch  : md-outputs
Author  : GitHub Actions <actions@github.com>
Time    : 2026-02-06 17:38:53 +0000
Message : markdown source builds

Auto-generated via `{sandpaper}`
Source  : 9663f6d
Branch  : main
Author  : Rabea Müller <42644523+RabeaMue@users.noreply.github.com>
Time    : 2026-02-06 17:38:09 +0000
Message : Merge pull request #66 from morskyjezek/main

Update text in 1.3 and multiple suggested revisions
github-actions bot pushed a commit that referenced this pull request Feb 24, 2026
Auto-generated via `{sandpaper}`
Source  : 09f24ec
Branch  : md-outputs
Author  : GitHub Actions <actions@github.com>
Time    : 2026-02-06 17:38:53 +0000
Message : markdown source builds

Auto-generated via `{sandpaper}`
Source  : 9663f6d
Branch  : main
Author  : Rabea Müller <42644523+RabeaMue@users.noreply.github.com>
Time    : 2026-02-06 17:38:09 +0000
Message : Merge pull request #66 from morskyjezek/main

Update text in 1.3 and multiple suggested revisions
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants