The Wayback Machine - https://web.archive.org/web/20201116180334/https://github.com/ryanmcdermott/clean-code-javascript/pull/251
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

Change wording and add examples for Liskov Substitution Principle #251

Open
wants to merge 1 commit into
base: master
from

Conversation

@strunkandwhite
Copy link

@strunkandwhite strunkandwhite commented Jan 15, 2019

Hello! Love this repo - incredibly useful, and I've bookmarked it for future reference.

I'd like to suggest a change to the writeup on the Liskov Substitution Principle. The text right now indicates that instances of a parent and child class can be used interchangeably. This is incorrect - the parent can be replaced by the child, but not the other way around. I've updated the wording and added a small illustrative example.

Thanks for maintaining this!

@ryanmcdermott
Copy link
Owner

@ryanmcdermott ryanmcdermott commented Jan 24, 2019

Thanks for your expertise! You're right.

Can you update your commit to not have the code example at the bottom. I like to keep the code examples only in a Good or Bad block

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
3 participants
You can’t perform that action at this time.