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]: Update README.md #47

Merged
merged 5 commits into from
Feb 10, 2024

Conversation

YASH-YADAV-dynamo
Copy link
Contributor

@YASH-YADAV-dynamo YASH-YADAV-dynamo commented Jan 16, 2024

Added commands in README.md for steps to set the environment for contribution


Description πŸ“œ

Fixed README.md with steps for user to just copy-paste in terminal to setup the environment.
Fixes <#45>


Domain of Contribution πŸ“Š

  • Documentation
  • Android (Kotlin)

Checklist βœ…

  • I follow Code of Conduct of this project.
  • I have performed a self-review of my own code or work.
  • I have put headings in my code, particularly in hard-to-understand areas.
  • My changes generates no new warnings.


Added commands in readme.md for steps to set the environment for contribution
Copy link
Owner

@vigneshshettyin vigneshshettyin left a comment

Choose a reason for hiding this comment

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

This is not enough. create a separate file with complete guide to setup project locally as well @YASH-YADAV-dynamo

@YASH-YADAV-dynamo
Copy link
Contributor Author

Ok,sure will update it.

@YASH-YADAV-dynamo
Copy link
Contributor Author

Created a separate file as "contribution.md" for steps to contribute and for setting up the project locally mentioned to install android studio

@YASH-YADAV-dynamo
Copy link
Contributor Author

If any more changes are needed ,please mention them in detail..i will surely work on them

@vigneshshettyin vigneshshettyin added documentation Improvements or additions to documentation Level1 GSSOC'22 Level 1 IWOC2024 labels Feb 10, 2024
@vigneshshettyin vigneshshettyin merged commit 29a3a3c into vigneshshettyin:main Feb 10, 2024
1 of 4 checks passed
@YASH-YADAV-dynamo YASH-YADAV-dynamo changed the title Update README.md [FIX:]Update README.md Feb 14, 2024
@YASH-YADAV-dynamo YASH-YADAV-dynamo changed the title [FIX:]Update README.md [FIX]:Update README.md Feb 14, 2024
@YASH-YADAV-dynamo YASH-YADAV-dynamo changed the title [FIX]:Update README.md FIX: Update README.md Feb 14, 2024
@YASH-YADAV-dynamo YASH-YADAV-dynamo changed the title FIX: Update README.md fix: Update README.md Feb 14, 2024
@YASH-YADAV-dynamo YASH-YADAV-dynamo changed the title fix: Update README.md [fix]: Update README.md Feb 14, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
documentation Improvements or additions to documentation IWOC2024 Level1 GSSOC'22 Level 1 title needs formatting
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

2 participants