-
Notifications
You must be signed in to change notification settings - Fork 3.8k
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
Adding [Term Entry] C++ Inheritance: Multiple Inheritance #6053
base: main
Are you sure you want to change the base?
Adding [Term Entry] C++ Inheritance: Multiple Inheritance #6053
Conversation
Hey @John-Kingham, please sign the CLA agreement using the link in the above comment. We won't be able to merge your contribution without that. Thanks. |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Hey @John-Kingham, thanks for contributing to docs. The entry looks good but I have a few suggestions. Please make the required changes so that we can then push this PR for a second review.
Let me know if you have any questions or doubts, will be happy to help. Happy contributing! 🚀
content/cpp/concepts/inheritance/terms/multiple-inheritance/multiple-inheritance.md
Show resolved
Hide resolved
content/cpp/concepts/inheritance/terms/multiple-inheritance/multiple-inheritance.md
Outdated
Show resolved
Hide resolved
content/cpp/concepts/inheritance/terms/multiple-inheritance/multiple-inheritance.md
Outdated
Show resolved
Hide resolved
content/cpp/concepts/inheritance/terms/multiple-inheritance/multiple-inheritance.md
Outdated
Show resolved
Hide resolved
content/cpp/concepts/inheritance/terms/multiple-inheritance/multiple-inheritance.md
Outdated
Show resolved
Hide resolved
content/cpp/concepts/inheritance/terms/multiple-inheritance/multiple-inheritance.md
Outdated
Show resolved
Hide resolved
content/cpp/concepts/inheritance/terms/multiple-inheritance/multiple-inheritance.md
Show resolved
Hide resolved
content/cpp/concepts/inheritance/terms/multiple-inheritance/multiple-inheritance.md
Outdated
Show resolved
Hide resolved
content/cpp/concepts/inheritance/terms/multiple-inheritance/multiple-inheritance.md
Show resolved
Hide resolved
content/cpp/concepts/inheritance/terms/multiple-inheritance/multiple-inheritance.md
Show resolved
Hide resolved
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Hey @John-Kingham, thanks for signing the CLA Agreement as per my last comment. This PR is now good for a second review.
Description
Adds a new term entry for
Multiple Inheritance
under C++. The entry is written in the new file:docs/content/cpp/concepts/inheritance/terms/multiple-inheritance/multiple-inheritance.md
. I'm doing this as part of the Codecademy Learn Git & GitHub course.Issue Solved
Closes #6019
Type of Change
Checklist
main
branch.Issues Solved
section.