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

Update teachingmaterial.tsv #9

Merged
merged 6 commits into from
Aug 18, 2024
Merged

Update teachingmaterial.tsv #9

merged 6 commits into from
Aug 18, 2024

Conversation

zackbatist
Copy link
Contributor

Adding a Spanish-language R tutorial. The coordinates are probably not placed properly.

Adding a Spanish-language R tutorial. The coordinates are probably not placed properly.
@nevrome
Copy link
Member

nevrome commented Aug 9, 2024

Cool! Thanks for the PR.

Unfortunately the validation fails with

Err "There was a problem on row 48, in the `Citation` field (column 17): I couldn't find the `Citation` column."

Probably there is a tab or maybe a newline missing somewhere. Could you check and try to make it valid?

Added missing columns.
Removing trailing new line
Re-adding dropped blank spaces as the end of lines with no citation value.
@zackbatist
Copy link
Contributor Author

Apologies for all the error notifications. I tried to revise the changes to produce valid tsv but I must be missing something that I can't pinpoint, since I never use this format.

@nevrome
Copy link
Member

nevrome commented Aug 18, 2024

Thanks for trying! There were just a couple of tabs missing in your new row, so that was easy to fix for me. I recommend editing .tsv (and .csv) files with LibreOffice Calc. That works really well and intuitively 👍

@nevrome nevrome merged commit 601dcb1 into main Aug 18, 2024
1 check passed
@nevrome nevrome deleted the zackbatist-patch-1 branch August 18, 2024 16:47
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