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

Improve Contribution Guide #1564

Closed
wants to merge 5 commits into from

Conversation

hendhamdi
Copy link

@hendhamdi hendhamdi commented Feb 1, 2025

Hello @maintainer, I've added a detailed contribution guide to help new contributors. Here's what I included:

  • Instructions for setting up the development environment.
  • Steps for submitting a PR.
  • Best practices for contributions.

I hope this meets your expectations. Please let me know if any changes are needed. Thank you! 😊

@sammyhori sammyhori linked an issue Feb 4, 2025 that may be closed by this pull request
CONTRIBUTING.md Outdated

5. **Push Your Changes**:
```bash
Copy

Choose a reason for hiding this comment

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

Why did you include 'Copy' in the Push Your Changes step?

Copy link
Author

@hendhamdi hendhamdi Feb 5, 2025

Choose a reason for hiding this comment

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

I'm sorry for the mistake, I will correct it promptly, and thank you for your feedback.

CONTRIBUTING.md Outdated
Comment on lines 11 to 13
```bash
git clone https://github.com/your-username/awesome-for-beginners.git
cd awesome-for-beginners
Copy link
Collaborator

Choose a reason for hiding this comment

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

For future reference, it is good practice to close your code blocks in markdown (using ```).

Copy link
Author

Choose a reason for hiding this comment

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

Thank you for the tip! As a beginner, I'm still learning Markdown syntax, and I appreciate your patience. I've fixed the code blocks by properly closing them with triple backticks (```).

Copy link
Collaborator

@sammyhori sammyhori left a comment

Choose a reason for hiding this comment

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

Thanks for the contribution.

Unfortunately I feel this contributing guide is not quite right for this repository.

Firstly, it does not include many of the previous guidelines, which state many of the repository specific steps that should be taken and requirements to be followed when considering a contribution on this repository.

Secondly, the guide is too complex, pursing a command line interface approach to cloning and opening a pull request: for those who need a guide for opening pull requests something more simple would be beneficial. On GitHub a user could reasonably easily contribute without leaving the browser, by going on the data.json file, pressing the edit button in the top right of the file, copying one of the yaml entries and filling it in with their own information. Then GitHub will guide them through making a pull request using their edit. We could even point people towards the first-contributions repo for a more comprehensive guide on contributing to a repository.

I do, however, like the idea of your issues section at the bottom, suggesting people open issues if they're not sure what to do or have a suggestion which they're not sure how to implement is a nice idea.

Copy link

This PR is not valid. Please read contributing guidelines.

@github-actions github-actions bot added the invalid Pull request is invalid label Feb 10, 2025
@github-actions github-actions bot closed this Feb 10, 2025
@hendhamdi
Copy link
Author

Hello @maintainer,

Thank you for your feedback! I've updated the CONTRIBUTING.md file to better align with the repository's needs. As a beginner, I really appreciate your guidance, and I've made the following changes:

Simplified the Contribution Process: The guide now focuses on contributing directly via GitHub's web interface, making it easier for beginners. I also added a direct link to the data.json file for quick access.

Kept the "Reporting an Issue" Section: I retained and improved this section, as you mentioned it was a good idea.

Added Additional Resources: I included a link to the First Contributions repository for users who want a more comprehensive guide.

Please let me know if there are any further adjustments needed. I'm happy to make additional changes to ensure the guide meets the repository's standards.

Thank you again for your patience and support! 😊

@sammyhori sammyhori reopened this Feb 11, 2025
Copy link

This PR is not valid. Please read contributing guidelines.

@github-actions github-actions bot closed this Feb 11, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
invalid Pull request is invalid
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Improve Contribution Guide
3 participants