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

Improved contribution guidelines #34

Merged
merged 4 commits into from
Oct 2, 2024
Merged

Conversation

J-B-Mugundh
Copy link
Contributor

@J-B-Mugundh J-B-Mugundh commented Oct 1, 2024

Purpose:
The previous contribution guidelines were difficult to follow, especially for new contributors. The guidelines have been improved with clearer instructions and GitHub commands to make them easier to understand and follow.

Impact:

  • Simplified the contribution process by providing step-by-step instructions.
  • Added GitHub commands to assist contributors in using the terminal effectively.
  • Ensured clarity around key tasks such as switching branches, forking, and submitting pull requests.
  • Encouraged contributors to engage with the project by starring the repository and following the organization.

Related Issue:
Resolves #27

@Harish-2003
Copy link
Contributor

Also mention to star the repo and follow the organisation

@Harish-2003
Copy link
Contributor

And why are u not writing an pr message you need to write what issue it closes and what does this pr does

@J-B-Mugundh
Copy link
Contributor Author

And why are u not writing an pr message you need to write what issue it closes and what does this pr does

I'm new to open source! I'll added them now

@Harish-2003
Copy link
Contributor

Also please mention the issue in the previous pr that was merged

@Harish-2003
Copy link
Contributor

And also mentioned some changes please look into it and make respective commits to the pr

@Harish-2003
Copy link
Contributor

And why are u not writing an pr message you need to write what issue it closes and what does this pr does

...

@J-B-Mugundh
Copy link
Contributor Author

And also mentioned some changes please look into it and make respective commits to the pr

I've mentioned to star the repo and follow our organization in the new commit!

@J-B-Mugundh
Copy link
Contributor Author

And why are u not writing an pr message you need to write what issue it closes and what does this pr does

...

I've edited the PR description and added in now!

@J-B-Mugundh
Copy link
Contributor Author

J-B-Mugundh commented Oct 1, 2024

Please ignore the last 2 commits. I thought the PR was merged since the changes were approved.

@Harish-2003 Harish-2003 added documentation Improvements or additions to documentation wob24 this is issue is part of winter of blockchain 2024 level1 labels Oct 2, 2024
@Harish-2003 Harish-2003 merged commit 42b4f3f into OpenTekHub:main Oct 2, 2024
1 check passed
@J-B-Mugundh
Copy link
Contributor Author

@Harish-2003 Could u please add gssoc-extd label to this?

@J-B-Mugundh
Copy link
Contributor Author

@sanjay-kv @Harish-2003 @KiranBaliga @Bruhbytes Could you please add gssoc-extd label here please!

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 level1 wob24 this is issue is part of winter of blockchain 2024
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Add detail instruction for Contribution guidelines
2 participants