Skip to content

Conversation

ndporter
Copy link
Contributor

Make the following types of changes throughout:

  1. typo corrections
  2. updates to lesson status (alpha, beta, stable)
  3. corrections to lists of tools

Make the following types of changes throughout:
1. typo corrections
2. updates to lesson status (alpha, beta, stable)
3. corrections to lists of tools
@netlify
Copy link

netlify bot commented Sep 25, 2025

Deploy Preview for carpentries-website ready!

Name Link
🔨 Latest commit b48d98c
🔍 Latest deploy log https://app.netlify.com/projects/carpentries-website/deploys/68d6956e82364a0009785630
😎 Deploy Preview https://deploy-preview-545--carpentries-website.netlify.app
📱 Preview on mobile
Toggle QR Code...

QR Code

Use your smartphone camera to open QR code link.

To edit notification comments on pull requests, go to your Netlify project configuration.

@maneesha
Copy link
Contributor

Thanks for going through this list @ndporter

Could we also use this opportunity to clean up a couple of other things?

  1. Lesson names are not consistent in the Carpentries mega-list and on each LP's website. For example, in this list, we refer to DC's "Intro to R and RStudio for Genomics" but on the DC site we call the same lesson "Data Analysis and Visualization in R"
  2. In this yaml file, I recognize the key "tool" is a bit ambiguous. I think the intent was to have that refer to the technology/language being taught and not the specific platform that uses that language. Just as we only list "R" and not "R Studio" we should only list "SQL" and not "SQLite". This part is something I noticed myself a week or two ago and was planning to open an issue about anyways.

I know these points may be bigger changes - and I know you're also doing this in some of your final acts in your role - so I am happy to merge this first and return to the other points later. Tagging @tobyhodges for review.

@maneesha maneesha requested a review from tobyhodges September 25, 2025 15:17
@tobyhodges
Copy link
Member

Thanks @ndporter for opening this, and @maneesha for finding the other inconsistencies. Regarding Maneesha's first point, I have opened datacarpentry/genomics-r-intro#318 to adjust the lesson title there. If and when that is merged, we can adjust here and make everything consistent. On the second point, let's update the tool listings here before merging.

Copy link
Member

@tobyhodges tobyhodges left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Happy for this to be merged after the listings of SQLite have been removed under tool

ndporter and others added 2 commits September 26, 2025 09:29
Co-authored-by: Toby Hodges <tbyhdgs@gmail.com>
@ndporter ndporter requested a review from tobyhodges September 26, 2025 13:30
@ndporter
Copy link
Contributor Author

Updated to remove all 4 SQLite tool tags

@maneesha maneesha closed this Sep 29, 2025
@maneesha maneesha reopened this Sep 29, 2025
@maneesha maneesha merged commit 9e3e2b0 into carpentries:main Sep 29, 2025
4 checks passed
@maneesha
Copy link
Contributor

Accidentally closed this without merging

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.

3 participants