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 Lesson24_Basic_Stats_IV_Significance.ipynb #51

Closed
wants to merge 3 commits into from

Conversation

echou89
Copy link
Contributor

@echou89 echou89 commented Jun 16, 2023

Fixed typos and added clarifications

"* `AdjLand`: Adjoining Land Purchase\n",
"* `Alloca`: Allocation - two linked properties with separate deeds, typically condo with a garage unit\n",
"* `Family`: Sale between family members\n",
"* `Partial`: Home was not completed when last assessed (associated with New Homes)",

Choose a reason for hiding this comment

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

Suggested change
"* `Partial`: Home was not completed when last assessed (associated with New Homes)",
"* `Partial`: Home was not completed when last assessed (associated with New Homes)"

This isn't a valid notebook because of the trailing ,. That's why the build is failing

@smith-kyle
Copy link

Using a notebook review tool can help out with these kinds of reviews: https://gitnotebooks.com/GWC-DCMB/curriculum-notebooks/pull/51

It's free for private repos so may be an easy win for these kinds of pull requests

@echou89 echou89 closed this Feb 28, 2024
@echou89 echou89 deleted the echou89-patch-1 branch February 28, 2024 00:25
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