Skip to content

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

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

Conversation

iambel
Copy link

@iambel iambel commented May 6, 2025

reorganized and emphasized key sections in CONTRIBUTING.md to improve clarity and guidance for new contributors

reorganized and emphasized key sections in CONTRIBUTING.md to improve clarity and guidance for new contributors
Copy link
Contributor

@karasowles karasowles 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 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
@iambel
Copy link
Author

iambel commented May 6, 2025

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

Copy link
Contributor

@adiati98 adiati98 left a 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! 👋
Copy link
Contributor

@adiati98 adiati98 May 7, 2025

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! 💚🚀
Copy link
Contributor

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> 💚
Copy link
Contributor

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.
Copy link
Contributor

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.

Suggested change
> **📌 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
Copy link
Contributor

@adiati98 adiati98 May 7, 2025

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.

Suggested change
git clone https://github.com/iambel/maintainermonth.git
git clone https://github.com/YOUR-USERNAME/maintainermonth.git

@iambel
Copy link
Author

iambel commented May 12, 2025

hey!
i reviewed all the feedback and pushed a new commit with some corrections. it should be cleaner now

@iambel
Copy link
Author

iambel commented May 19, 2025

hi! just following up on this PR. let me know if there's anything i should improve or clarify. 👐

Copy link
Contributor

@adiati98 adiati98 left a 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. 👍🏼

Comment on lines +16 to +17
- [Adding a New Resource
](#adding-a-new-resource)
Copy link
Contributor

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.

Suggested change
- [Adding a New Resource
](#adding-a-new-resource)
- [Adding a New Resource](#adding-a-new-resource)

Comment on lines +19 to +20
- [Making Code Changes
](#making-code-changes)
Copy link
Contributor

Choose a reason for hiding this comment

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

Suggested change
- [Making Code Changes
](#making-code-changes)
- [Making Code Changes](#making-code-changes)

Comment on lines +22 to +23
- [Setting Up Your Environment
](#setting-up-your-enviroment)
Copy link
Contributor

Choose a reason for hiding this comment

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

Suggested change
- [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
Copy link
Contributor

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.

Suggested change
### Adding a New Event
### Adding a New Event

Comment on lines +46 to +51
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:
Copy link
Contributor

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
Copy link
Contributor

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.

Suggested change
## Frontmatter Fields
### Frontmatter Fields


#### Event Fields
## Event Fields
Copy link
Contributor

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.

Suggested change
## 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
Copy link
Contributor

Choose a reason for hiding this comment

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

Suggested change
## 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
Copy link
Contributor

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.

Suggested change
## 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
Copy link
Contributor

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.

Suggested change
## Questions
## Questions?

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants