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

grammatical/typo errors fixed #54

Open
wants to merge 3 commits into
base: master
Choose a base branch
from

Conversation

varshitha1707
Copy link

@varshitha1707 varshitha1707 commented Jun 20, 2021

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.

@@ -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.",

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"

Copy link
Author

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

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.

Copy link
Author

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.

@varshitha1707
Copy link
Author

varshitha1707 commented Jun 28, 2021

@michaelskyba could you merge it to master please?

@michaelskyba
Copy link

I don't have write access to this repo; my only significance is that I own the original issue.

inthedownpour added a commit to inthedownpour/web-skills that referenced this pull request Sep 6, 2021
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.

Typos
2 participants