search

1Shubham7 / Wife-Predictor-v2

This is a fun project that predicts your wife. The web app has been deployed on Netlify.
https://wife-predictor.netlify.app/
MIT License
26 stars 27 forks source link

Remove the grammatical mistakes in the website. #6

Closed 1Shubham7 closed 1 year ago

1Shubham7 commented 1 year ago

The website and the readme.md file has a lot of grammatical mistakes. Please correct them.

theQuery commented 1 year ago

I'd like to be assigned this task

1Shubham7 commented 1 year ago

ok sure.

theQuery commented 1 year ago

Thank you!

1Shubham7 commented 1 year ago

@theQuery , I can't merge your PR, it is too big. The website will go offline. Please make small changes and do small PR. Would you want to do it again.

theQuery commented 1 year ago

How small should the PRs be? One per file?

1Shubham7 commented 1 year ago

yup.

theQuery commented 1 year ago

I believe it should be okay to merge them all at once, though? If it's deployed on Netlify as it says that it is, then there shouldn't be any downtime to worry about.

1Shubham7 commented 1 year ago

@theQuery . Yup , you are right. It had some conflicts but I resolved them. Thank you for contributing. Do contribute more. your contributions are highly appreciated.

1Shubham7 commented 1 year ago

Do give the repo a star!!

theQuery commented 1 year ago

It's my pleasure! I'll make sure to star it as well