Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Question about 02-match-extract-strings.md #220

Open
doujouDC opened this issue Mar 13, 2024 · 1 comment
Open

Question about 02-match-extract-strings.md #220

doujouDC opened this issue Mar 13, 2024 · 1 comment

Comments

@doujouDC
Copy link
Contributor

doujouDC commented Mar 13, 2024

How could the content be improved?

Hi, I am teaching this lesson for the first time next month and i want to make sure I am not missing someone in this section: Taking spaces into consideration
Type community . You get three matches. Why not six?

The solution says "The string 'community-led' matches the first search, but drops out of this result because the space does not match the character -." but we are asked to type in "community", not "community-led". Am I missing something?

Actually, I think I figured this, it was not clear that you needed to type community with a space. I will mention this when teaching the lesson

Which part of the content does your suggestion apply to?

https://github.com/LibraryCarpentry/lc-data-intro/blob/main/episodes/02-match-extract-strings.md

@doujouDC
Copy link
Contributor Author

Hey, I think I understand the issue that I had with this section a little better. However, I did notice that the link in the following sections no longer works: "Find the latest list here. What expression would capture this? Hint: the . is also a metacharacter, so you will have to use the escape \ to express a literal period. Note: for the string after the period, we did not try to match a - character, since those rarely appear in the characters after the period at the end of an email address." This is on the following page: https://librarycarpentry.org/lc-data-intro/02-match-extract-strings.html#exercise-finding-email-addresses-using-regex101.com

aforestsomewhere added a commit to aforestsomewhere/lc-data-intro that referenced this issue Apr 27, 2024
episodes\01-regular-expressions.md
- Elevated "Learning common regex metacharacters" to improve the layout of the episode
- Numbered challenges / exercises (LibraryCarpentry#198)
- Removed decayed link, replaced with regex100.com library of commmunity-submitted regex (LibraryCarpentry#210)

episodes\01-regular-expressions.md
- Numbered exercises (LibraryCarpentry#198)
- Clarified that learner should type add a space after community for first challenge in exercise 2.1 (LibraryCarpentry#220 (comment))
- Added exercise 2.4 on use of regex in R (self-organisied workshop, this lessson is taught after https://datacarpentry.org/r-socialsci/)
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

No branches or pull requests

1 participant