Skip to content

adding a CONTRIBUTING.md file - #868

Open
Amrit-raj50 wants to merge 7 commits into
git:masterfrom
Amrit-raj50:add-contributing-md
Open

Amrit-raj50 wants to merge 7 commits into
git:masterfrom
Amrit-raj50:add-contributing-md

Conversation

@Amrit-raj50

@Amrit-raj50 Amrit-raj50 commented Sep 21, 2026

Copy link
Copy Markdown
Contributor

This PR adds a CONTRIBUTING.md file to the repository.

GitHub automatically surfaces this file to anyone who opens a new
issue or pull request, helping new contributors understand how to
participate before they submit anything.

The file covers:

  • Ways to contribute (typos, content, newsletter, infrastructure)
  • Submitting changes via Pull Request (step-by-step)
  • Submitting changes via email patch (with maintainer contacts)
  • Running the site locally (Jekyll setup)
  • Writing guidelines (Markdown style, code fences)
  • Code style notes (JS, CSS, Gemfile, whitespace)
  • Git Rev News newsletter contribution process
  • Reporting issues (what to include in a bug report)

All information is based on the existing README.md and repository
conventions — nothing new is introduced, just consolidated in one
place for contributor discoverability.

Closes/Relates to: (add issue number if one exists)

issue : #864

@Amrit-raj50

Copy link
Copy Markdown
Contributor Author

@chriscool @sivaraam — Could you take a look at this when you get
a chance?

Comment thread CONTRIBUTING.md
@@ -0,0 +1,144 @@
# Contributing to Git Developer Pages

Thank you for your interest in contributing to [git.github.io](https://git.github.io) — the community website for Git development, home to the [Git Rev News newsletter](https://git.github.io/rev_news/), [Hacking Git](https://git.github.io/Hacking-Git/), and mentoring program information.

Copy link
Copy Markdown
Collaborator

Choose a reason for hiding this comment

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

Please avoid long lines like this. Instead please split them at around 72 characters.

Also please avoid characters. It looks too much AI generated.

Comment thread CONTRIBUTING.md
---

## Ways to Contribute

Copy link
Copy Markdown
Collaborator

Choose a reason for hiding this comment

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

Please add an introduction sentence, like "In general we accept all kind of contributions to all the pages on this website, but here are some suggestions along with a few guidelines:"

Comment thread CONTRIBUTING.md

## Ways to Contribute

- Fix typos, broken links, or outdated information

Copy link
Copy Markdown
Collaborator

Choose a reason for hiding this comment

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

Those kinds of contributions are fine for all the website except already published editions of Git Rev News. I don't think we want to fix things in old editions.

@jnareb, @mjaix and @sivaraam do you have an opinion on this?

Comment thread CONTRIBUTING.md
## Ways to Contribute

- Fix typos, broken links, or outdated information
- Improve or add pages under `links/`, `rev_news/`, or other sections

Copy link
Copy Markdown
Collaborator

Choose a reason for hiding this comment

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

I am not sure listing those directories make sense. I think it would make more sense to list the section on the website instead.

Comment thread CONTRIBUTING.md

- Fix typos, broken links, or outdated information
- Improve or add pages under `links/`, `rev_news/`, or other sections
- Add entries to the Git Rev News newsletter

@chriscool chriscool Sep 22, 2026

Copy link
Copy Markdown
Collaborator

Choose a reason for hiding this comment

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

It's not clear what "entries" means here. I think this should be more detailed. It should say what could be added in which section, how to find the upcoming edition where things can be added, and so on.

Comment thread CONTRIBUTING.md
- Fix typos, broken links, or outdated information
- Improve or add pages under `links/`, `rev_news/`, or other sections
- Add entries to the Git Rev News newsletter
- Improve site infrastructure (CSS, JavaScript, Jekyll layouts/plugins)

Copy link
Copy Markdown
Collaborator

Choose a reason for hiding this comment

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

For any big improvement, people should look at existing issues and if needed create a new one before starting to work.

Comment thread CONTRIBUTING.md
- Add entries to the Git Rev News newsletter
- Improve site infrastructure (CSS, JavaScript, Jekyll layouts/plugins)
- Report issues via [GitHub Issues](https://github.com/git/git.github.io/issues)

Copy link
Copy Markdown
Collaborator

Choose a reason for hiding this comment

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

They could also give their opinion on existing issues or review existing MRs.

Comment thread CONTRIBUTING.md
git checkout -b my-fix
```
4. **Make your changes**, following the [Writing Guidelines](#writing-guidelines) below.
5. **Test locally** by [running the site](#running-the-site-locally) and verifying your changes look correct.

Copy link
Copy Markdown
Collaborator

Choose a reason for hiding this comment

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

It's not necessary to test their changes locally when contributing to a Git Rev News edition.

Comment thread CONTRIBUTING.md
6. **Commit** with a clear, descriptive message:
```sh
git add .
git commit -m "Brief description of the change"

Copy link
Copy Markdown
Collaborator

Choose a reason for hiding this comment

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

When working on a Git Rev News edition, the subject should start with "rn-XXX: " where XXX is the edition you are working on.

Comment thread CONTRIBUTING.md
If you want to contribute to a newsletter edition:
- Check the [news sources](https://git.github.io/rev_news/news_sources/) page for submission guidelines.
- Contact the maintainers via email (see [above](#submitting-changes-via-email)).

Copy link
Copy Markdown
Collaborator

Choose a reason for hiding this comment

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

There is an issue created for each edition. It's best to contact us on this issue if you want to contribute to an edition.

@chriscool

Copy link
Copy Markdown
Collaborator

@Amrit-raj50 I have left a number of comments. Thanks.

@Amrit-raj50

Copy link
Copy Markdown
Contributor Author

I will raise the new PR for this after thrusday

This branch has not been deployed

No deployments
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.

2 participants