Skip to content
This repository has been archived by the owner on Oct 7, 2021. It is now read-only.

Create 188.json : Added some personal favourite quotes. #14

Merged
merged 2 commits into from
Sep 30, 2020
Merged

Create 188.json : Added some personal favourite quotes. #14

merged 2 commits into from
Sep 30, 2020

Conversation

swapniljha001
Copy link

Check one of the below boxes as per your PR

Make sure these boxes are checked before your pull request (PR) is ready to be reviewed and merged. Thanks!

  • PR is descriptively titled 📑 and links the original issue above 🔗
  • code is in uniquely-named branch and has no merge conflicts 📁
  • screenshots/GIFs are attached 📎 if applicable
  • ask @avats-dev for help, in a comment below

I'm happy to help you get this ready -- don't be afraid to ask for help, and don't be discouraged !

Thanks!

Added some personal favourite quotes.
Copy link
Owner

@avats-dev avats-dev left a comment

Choose a reason for hiding this comment

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

Nice work @swapniljha001 🎉 and thanks for your contribution.
As 188.json have been merged recently, there are merge conflicts now. And as you've opened #13 also, can you merge these both files in 187.json or alternatively you can create a new one 190.json for this one.
Thanks.

Renamed the file as mentioned in #14 (review) in #14.
@swapniljha001
Copy link
Author

@avats-dev I have changed the filename to 190.json to resolve the conflict issues.

@avats-dev avats-dev merged commit 6c4c966 into avats-dev:master Sep 30, 2020
@swapniljha001 swapniljha001 deleted the patch-2 branch October 1, 2020 02:50
@swapniljha001
Copy link
Author

Thanks, @avats-dev for accepting the PR 👍🏻

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants