-
Notifications
You must be signed in to change notification settings - Fork 45
docs: improve CONTRIBUTING.md #327
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
base: main
Are you sure you want to change the base?
Conversation
reorganized and emphasized key sections in CONTRIBUTING.md to improve clarity and guidance for new contributors
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Thanks for adding some clarity here and streamlining it!
I'd like to keep the "title: 'Your Event Title' etc. sample code, rather than put the Africa example back in. It's been easier to copy-paste and fill in.
Let's also keep the Content Guidelines at the bottom, for thoroughness.
correcting some details based on feedback received
i noticed what was missing and submitted a pull request with changes based on your feedback (thanks!). let me know if there’s anything else you would like me to adjust |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Hey @iambel,
You made such a great start here! ✨
I left some comments here for my thoughts and ideas. However, please note that I'm not a maintainer of this repo. So, all decisions will go to the maintainers. 👍🏼
Other than these comments, I also have some thoughts in general.
-
I see there are many icons here. The titles also have an icon. I'm not sure if a guide should have icons for titles, but if this is what the maintainers want, then let's go with it. 🙂
-
There are many bold sentences. Highlighting things important with bold fonts are great, but when there are too many, I think it would make folks to pay more attention to the bold ones and screen the whole guide rather than reading it thoroughly.
cc: @karasowles
CONTRIBUTING.md
Outdated
@@ -1,30 +1,54 @@ | |||
# Contributing to GitHub Maintainer Month | |||
|
|||
:+1::tada: First off, thanks for taking the time to contribute! :tada::+1: | |||
### Hey there! 👋 |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
suggestions: Markdown best practice and accessibility wise, there should be an H2 before H3 after the H1. Is the reason for applying H3 here because of the font size? Because we better make this an H2 or part of the paragraph instead.
CONTRIBUTING.md
Outdated
## Quick Navigation | ||
This document provides **guidelines** and **instructions** for contributing to the **Maintainer Month** website. | ||
|
||
🎇 Don't worry if you're new here, we appreciate supporting and giving chances for **first-time contributions**! If you have any questions, **feel free to reach out for help** and we'll try to answer as soon as possible! 💚🚀 |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
questions: I believe we want to encourage first-time contributor with this sentence? However, I get the impression that we only receiving first-time contribution. Or is this the intention?
suggestion: We might want to reword this to welcoming all contributors (first timer and regular) if that's the goal.
CONTRIBUTING.md
Outdated
|
||
The Maintainer Month website is maintained by the Open Source team at GitHub. You can reach out at <maintainermonth @ github .com> | ||
The **Maintainer Month** website is maintained by the **Open Source team at GitHub**. You can reach out at <maintainermonth@github.com> 💚 |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
@karasowles now there is a CODEOWNERS
file. Do you want folks to reach out to the email for questions and all, or tag the maintainers that are listed in the CODEOWNERS
?
CONTRIBUTING.md
Outdated
|
||
> **Important:** Do not modify variable names in frontmatter sections (between `---`). These are required for the website to function properly. | ||
> **📌 Important:** Do not modify variable names in frontmatter sections (between `---`). These are required for the website to function properly. |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
suggestions: Maybe we can utilize the GitHub admonition here.
> **📌 Important:** Do not modify variable names in frontmatter sections (between `---`). These are required for the website to function properly. | |
> [!IMPORTANT] | |
> Do not modify variable names in frontmatter sections (between `---`). These are required for the website to function properly. |
CONTRIBUTING.md
Outdated
|
||
1. Clone the repository | ||
|
||
```bash | ||
git clone https://github.com/iambel/maintainermonth.git |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
suggestion: I believe we want this guide to be general and easier to understand. So, let's do GitHub's template (refer to the step 6) here.
git clone https://github.com/iambel/maintainermonth.git | |
git clone https://github.com/YOUR-USERNAME/maintainermonth.git |
hey! |
hi! just following up on this PR. let me know if there's anything i should improve or clarify. 👐 |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Hey @iambel,
I left some reviews here.
As for content matters, I leave it to the maintainers to review them. 👍🏼
- [Adding a New Resource | ||
](#adding-a-new-resource) |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
The closing bracket should be inline.
- [Adding a New Resource | |
](#adding-a-new-resource) | |
- [Adding a New Resource](#adding-a-new-resource) |
- [Making Code Changes | ||
](#making-code-changes) |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
- [Making Code Changes | |
](#making-code-changes) | |
- [Making Code Changes](#making-code-changes) |
- [Setting Up Your Environment | ||
](#setting-up-your-enviroment) |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
- [Setting Up Your Environment | |
](#setting-up-your-enviroment) | |
- [Setting Up Your Environment](#setting-up-your-enviroment) |
|
||
## Contribution Walkthrough | ||
|
||
### Adding a New Event | ||
### → Adding a New Event |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
As there is no icon for the titles anymore, let's remove this one as well.
### → Adding a New Event | |
### Adding a New Event |
If you want to add a new event, go to the `content/events/` folder. | ||
|
||
1. Navigate to the `content/events/` folder | ||
2. Create a new markdown file with a descriptive name (e.g., `2025-05-20-your-event-name.md`) | ||
3. Use the following template: | ||
Each event is documented in a markdown file for each one. So, to create one: | ||
|
||
2. Create a new markdown file with a descriptive name (e.g., `2025-05-20-your-event-name.md` ) | ||
3. Use the following template to add the event details: |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Thoughts: I think we can stick to the original instructions here. But maintainers are the best to decide.
1. Navigate to the `content/events/` folder
2. Create a new markdown file with a descriptive name (e.g., `2025-05-20-your-event-name.md`)
3. Use the following template:
...
### Frontmatter Fields | ||
Here you’ll find explanations and instructions for filling out content templates. Following these guidelines helps keep everything organized and easy to review! | ||
|
||
## → Frontmatter Fields |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
This is part of "Content Guidelines" section. So, it should be an H3.
## → Frontmatter Fields | |
### Frontmatter Fields |
|
||
#### Event Fields | ||
## → Event Fields |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
This is part of "Frontmatter Fields". Therefore, it should be an H4.
## → Event Fields | |
#### Event Fields |
@@ -123,7 +164,7 @@ Detailed description of your event goes here. You can use markdown formatting. | |||
- `userLink`: Link to organizer profile/website | |||
- `linkUrl`: Direct link to the event | |||
|
|||
#### Library Resource Fields | |||
## → Library Resource Fields |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
## → Library Resource Fields | |
#### Library Resource Fields |
@@ -132,7 +173,7 @@ Detailed description of your event goes here. You can use markdown formatting. | |||
- `type`: Content type (e.g., `video`, `article`, `tutorial`) | |||
- `topics`: Relevant topic tags | |||
|
|||
### Static Content | |||
## → Static Content |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
This is part of "Content Guidelines". So, it should be an H3.
## → Static Content | |
### Static Content |
@@ -141,6 +182,6 @@ Other content files are organized as follows: | |||
|
|||
When editing these files, maintain the existing structure and frontmatter fields. | |||
|
|||
### Questions? | |||
## Questions |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Let's put back the questions mark here. Without one, it feels like we are the one who have questions for folks.
## Questions | |
## Questions? |
reorganized and emphasized key sections in
CONTRIBUTING.md
to improve clarity and guidance for new contributors