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.

Sign up for GitHub

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

Jump to bottom

Fix issue 124 bad null vals table #132

Merged
merged 4 commits into from Dec 3, 2022
Merged

Fix issue 124 bad null vals table #132

merged 4 commits into from Dec 3, 2022

Conversation

morskyjezek
Copy link
Contributor

@morskyjezek morskyjezek commented Dec 1, 2022 •
edited

This branch is intended to address the issues in issue #124 .

Changes:

  • update null value table to text, to improve accessibility of the information
  • correct citation to the source article for the null value suggestions
  • update and revise the problem/solution text for the null value example to better match the versions in DataCarpentry version of the lesson.

@morskyjezek morskyjezek mentioned this pull request Dec 1, 2022
Closed
@hoytpr hoytpr mentioned this pull request Dec 2, 2022
Merged
@morskyjezek morskyjezek merged commit a94a5fa into gh-pages Dec 3, 2022
@morskyjezek morskyjezek deleted the fix-issue-124-bad-null-vals-table branch December 3, 2022 00:05
zkamvar pushed a commit that referenced this pull request Apr 21, 2023
@morskyjezek
Merge pull request #132 from LibraryCarpentry/fix-issue-124-bad-null-…
4fd60a2 
…vals-table

Fix issue 124 bad null vals table
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@tobyhodges tobyhodges tobyhodges approved these changes

Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.

None yet
2 participants
@morskyjezek @tobyhodges