-
Notifications
You must be signed in to change notification settings - Fork 654
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
grammatical/typo errors fixed #54
base: master
Are you sure you want to change the base?
Conversation
@@ -7,7 +7,7 @@ export const architectureCollection = { | |||
{ | |||
name: "Programming Paradigms", | |||
description: { | |||
text: "Learn how programming languages can be categoried into different paraigmes based on their features.", | |||
text: "Learn how programming languages can be categoried into different paradigmes based on their features.", |
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.
Don't forget to remove the "e" in "paradigmes"
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.
I've made the changes. Please check
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.
That looks good, although I think you forgot:
Progressive Webapps > Security > Clickjacking
Change "these kind of" to "these kinds of".
Also, README.md has these same errors, so I guess the changes should be copied over to there as well. It doesn't necessarily have to be part of the same PR, though.
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.
I have done the change, and as for the README.md I'll create another issue which can be done with a new PR.
@michaelskyba could you merge it to master please? |
I don't have write access to this repo; my only significance is that I own the original issue. |
Junior를 Develop에 merge(andreasbm#53)
Reference Issues:
Fixes #52 TYPOS
Changes:
I have fixed all the typos and grammatical errors which were visible and enhanced your code a little bit.
I have also solved/ corrected all the given changes in the ISSUE NO. #52
Thanks for making Web Skills, it has been a great resource and really helpful! I was looking for ways to contribute to this wonderfull project by doing something at least.
This PR has been raised under LGMSOC21
Thankyou.