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

Fix typos in README #236

Merged
merged 10 commits into from
Sep 10, 2020
Merged

Fix typos in README #236

merged 10 commits into from
Sep 10, 2020

Conversation

lARSHADl
Copy link
Contributor

No description provided.

@vercel
Copy link

vercel bot commented Aug 27, 2020

This pull request is being automatically deployed with Vercel (learn more).
To see the status of your deployment, click below or on the icon next to each commit.

README.md Outdated
@@ -99,9 +99,9 @@ Note: Instead of creating a .env file, you can just run this command "cp env.exa
REACT_APP_GITHUB_TOKEN = "YOUR GITHUB TOKEN HERE"
```

Set `showGithubProfile` to true or false to show Contact Profile using Github, defaults to false.
Set `showGithubProfile` to true or false to show Contact Profile using Github,Default is set as false.
Copy link
Collaborator

Choose a reason for hiding this comment

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

I think the previous one correct.

Copy link
Collaborator

Choose a reason for hiding this comment

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

@lARSHADl Could you revert this to how it was before?

@lARSHADl lARSHADl requested a review from naveen521kk August 28, 2020 13:57
Copy link
Collaborator

@naveen521kk naveen521kk left a comment

Choose a reason for hiding this comment

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

Sorry for delay :(

README.md Outdated

Note: Open Source Projects section only show pinned itens of your Github. If you are seeing [this](https://i.imgur.com/Hj6mu1K.png), please follow this
Note: Open Source Projects section only show pinned items of your Github. If you are seeing [this](https://i.imgur.com/Hj6mu1K.png), please follow this
Copy link
Collaborator

Choose a reason for hiding this comment

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

Suggestion: Embed the image instead of saying, if you see this.

README.md Outdated

Note: Open Source Projects section only show pinned itens of your Github. If you are seeing [this](https://i.imgur.com/Hj6mu1K.png), please follow this
Note: Open Source Projects section only show pinned items of your Github. If you are seeing [this](https://i.imgur.com/Hj6mu1K.png), please follow this
Copy link
Collaborator

Choose a reason for hiding this comment

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

Change follow this to follow these.

README.md Outdated

Note: Open Source Projects section only show pinned itens of your Github. If you are seeing [this](https://i.imgur.com/Hj6mu1K.png), please follow this
Note: Open Source Projects section only show pinned items of your Github. If you are seeing [this](https://i.imgur.com/Hj6mu1K.png), please follow this
[Instructions](https://docs.github.com/en/enterprise/2.13/user/articles/pinning-items-to-your-profile).
Copy link
Collaborator

Choose a reason for hiding this comment

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

Remove capitalized I in Instructions.

@naveen521kk
Copy link
Collaborator

Any update on this @lARSHADl ?

@lARSHADl
Copy link
Contributor Author

lARSHADl commented Sep 7, 2020

Sorry guys new to github and this was my first commit

@lARSHADl
Copy link
Contributor Author

lARSHADl commented Sep 7, 2020 via email

README.md Outdated
Comment on lines 105 to 108


<img src = "https://i.imgur.com/Hj6mu1K.png" alt = " Error" >

Copy link
Collaborator

Choose a reason for hiding this comment

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

Why not use markdown like

Suggested change
<img src = "https://i.imgur.com/Hj6mu1K.png" alt = " Error" >
![Error](https://i.imgur.com/Hj6mu1K.png)

README.md Outdated

<img src = "https://i.imgur.com/Hj6mu1K.png" alt = " Error" >

Please follow these
[Instructions](https://docs.github.com/en/enterprise/2.13/user/articles/pinning-items-to-your-profile).
Copy link
Collaborator

Choose a reason for hiding this comment

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

As @kartikcho said do

Suggested change
[Instructions](https://docs.github.com/en/enterprise/2.13/user/articles/pinning-items-to-your-profile).
[instructions](https://docs.github.com/en/enterprise/2.13/user/articles/pinning-items-to-your-profile).

@lARSHADl
Copy link
Contributor Author

lARSHADl commented Sep 7, 2020

Will change to the last check point with ! Markdown .

@kartikcho kartikcho changed the title Update README.md Fix typos in README Sep 8, 2020
Copy link
Collaborator

@kartikcho kartikcho left a comment

Choose a reason for hiding this comment

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

Hey @lARSHADl, thanks for opening a PR for this, everything looks good now!

@lARSHADl
Copy link
Contributor Author

Is there anything that's bothering you, please feel free to suggest ?

Copy link
Collaborator

@naveen521kk naveen521kk left a comment

Choose a reason for hiding this comment

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

Sorry for the delay!

@naveen521kk naveen521kk merged commit ebdaa5e into saadpasta:master Sep 10, 2020
@naveen521kk
Copy link
Collaborator

@all-contributors please add @lARSHADl for doc

@allcontributors
Copy link
Contributor

@naveen521kk

I've put up a pull request to add @lARSHADl! 🎉

Mehranmzn pushed a commit to Mehranmzn/mehranmzn.github.io that referenced this pull request Sep 13, 2024
* Update README.md

* Correction

* Image added

* Added space

* Image added

* Update Readme

* Update READ.md

* new

* Fix typos in READ.md

* Change some lingo

Co-authored-by: Kartik Choudhary <kartikch918@gmail.com>
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