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

docs: clear and concised reading #52

Open
wants to merge 2 commits into
base: main
Choose a base branch
from

Conversation

07tAnYa
Copy link

@07tAnYa 07tAnYa commented Oct 14, 2023

What does this PR do?

This PR modifies the contributing.md file.

Summary of changes:
Corrected grammar and spelling errors
Made the language more clear and concise
Added screenshots

Fixes #11

Requirement/Documentation

  • If there is a requirement document, please, share it here. NA
  • If there is ab UI/UX design document, please, share it here. NA

Type of change

  • Bug fix (non-breaking change which fixes an issue)
  • New feature (non-breaking change which adds functionality)

How should this be tested?

  • Are there environment variables that should be set? NO
  • What are the minimal test data to have? NA
  • What is expected (happy path) to have (input and output)? NA
  • Any other important info that could help to test that PR? NO

Mandatory Tasks

  • Make sure you have self-reviewed the code. A decent size PR without self-review might be rejected.

@github-actions github-actions bot added bug Something isn't working documentation Improvements or additions to documentation good first issue Good for newcomers hacktoberfest Hacktoberfest Accepted Issues labels Oct 14, 2023
@github-actions
Copy link

github-actions bot commented Oct 14, 2023

Thank you for following the naming conventions! 🙏

@07tAnYa 07tAnYa changed the title Feature/clear and concised reading feat: clear and concised reading Oct 14, 2023
@sujayshanbhag
Copy link

This isn't a feature. You should have prefixed it as docs. A feature would be a new functionality in the app. You can just edit it. Hope this helps.

@07tAnYa
Copy link
Author

07tAnYa commented Oct 16, 2023

This isn't a feature. You should have prefixed it as docs. A feature would be a new functionality in the app. You can just edit it. Hope this helps.

okay : )

@07tAnYa 07tAnYa changed the title feat: clear and concised reading docs: clear and concised reading Oct 16, 2023
@07tAnYa
Copy link
Author

07tAnYa commented Oct 16, 2023

@piyushgarg-dev Sir kindly review my PR .

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug Something isn't working documentation Improvements or additions to documentation good first issue Good for newcomers hacktoberfest Hacktoberfest Accepted Issues
Projects
None yet
Development

Successfully merging this pull request may close these issues.

[Docs] Correct the Grammar of CONTRIBUTING.md
2 participants