-
Notifications
You must be signed in to change notification settings - Fork 325
adding a CONTRIBUTING.md file #868
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: master
Are you sure you want to change the base?
Changes from all commits
3a67334
73e9b58
bea6152
530664a
ebcae72
30a7192
3dae779
File filter
Filter by extension
Conversations
Jump to
Diff view
Diff view
There are no files selected for viewing
| Original file line number | Diff line number | Diff line change |
|---|---|---|
| @@ -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. | ||
|
|
||
| --- | ||
|
|
||
| ## Table of Contents | ||
|
|
||
| - [Ways to Contribute](#ways-to-contribute) | ||
| - [Submitting Changes via Pull Request](#submitting-changes-via-pull-request) | ||
| - [Submitting Changes via Email](#submitting-changes-via-email) | ||
| - [Running the Site Locally](#running-the-site-locally) | ||
| - [Writing Guidelines](#writing-guidelines) | ||
| - [Code Style](#code-style) | ||
| - [Git Rev News Newsletter](#git-rev-news-newsletter) | ||
| - [Reporting Issues](#reporting-issues) | ||
|
|
||
| --- | ||
|
|
||
| ## Ways to Contribute | ||
|
|
||
|
Collaborator
There was a problem hiding this comment. Choose a reason for hiding this commentThe 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:" |
||
| - Fix typos, broken links, or outdated information | ||
|
Collaborator
There was a problem hiding this comment. Choose a reason for hiding this commentThe reason will be displayed to describe this comment to others. Learn more. |
||
| - Improve or add pages under `links/`, `rev_news/`, or other sections | ||
|
Collaborator
There was a problem hiding this comment. Choose a reason for hiding this commentThe 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. |
||
| - Add entries to the Git Rev News newsletter | ||
|
Collaborator
There was a problem hiding this comment. Choose a reason for hiding this commentThe 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. |
||
| - Improve site infrastructure (CSS, JavaScript, Jekyll layouts/plugins) | ||
|
Collaborator
There was a problem hiding this comment. Choose a reason for hiding this commentThe 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. |
||
| - Report issues via [GitHub Issues](https://github.com/git/git.github.io/issues) | ||
|
|
||
|
Collaborator
There was a problem hiding this comment. Choose a reason for hiding this commentThe 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. |
||
| --- | ||
|
|
||
| ## Submitting Changes via Pull Request | ||
|
|
||
| 1. **Fork** this repository on GitHub. | ||
| 2. **Clone** your fork locally: | ||
| ```sh | ||
| git clone https://github.com/<your-username>/git.github.io.git | ||
| cd git.github.io | ||
| ``` | ||
| 3. **Create a branch** for your changes: | ||
| ```sh | ||
| 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. | ||
|
Collaborator
There was a problem hiding this comment. Choose a reason for hiding this commentThe 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. |
||
| 6. **Commit** with a clear, descriptive message: | ||
| ```sh | ||
| git add . | ||
| git commit -m "Brief description of the change" | ||
|
Collaborator
There was a problem hiding this comment. Choose a reason for hiding this commentThe 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. |
||
| ``` | ||
| 7. **Push** to your fork and open a Pull Request against the `master` branch of this repository. | ||
|
|
||
| > **Note:** Keep PRs focused on a single topic. Avoid mixing unrelated changes (e.g., infrastructure changes with content updates). | ||
|
|
||
| --- | ||
|
|
||
| ## Submitting Changes via Email | ||
|
|
||
| If you prefer not to use GitHub, you can send patches by email to the maintainers: | ||
|
|
||
| - **Christian Couder** — <christian.couder@gmail.com> | ||
| - **Kaartic Sivaraam** — <kaartic.sivaraam+git@gmail.com> | ||
|
|
||
| You may also CC the Git mailing list at <git@vger.kernel.org> if appropriate. | ||
|
|
||
| --- | ||
|
|
||
| ## Running the Site Locally | ||
|
|
||
| This site is built with [Jekyll](https://jekyllrb.com/) and hosted via GitHub Pages. | ||
|
|
||
| ### Prerequisites | ||
|
|
||
| - Ruby (2.x or later) with development packages | ||
| - Bundler | ||
|
|
||
| ### Steps | ||
|
|
||
| ```sh | ||
| # Install system dependency (Ubuntu/Debian) | ||
| sudo apt-get install zlib1g-dev | ||
|
|
||
| # Install Bundler gem | ||
| sudo gem install bundler | ||
|
|
||
| # Clone the repo and install dependencies | ||
| git clone https://github.com/git/git.github.io.git | ||
| cd git.github.io | ||
| bundle install | ||
|
|
||
| # Start the local server | ||
| bundle exec jekyll serve | ||
| ``` | ||
|
|
||
| Then open your browser at <http://localhost:4000>. | ||
|
|
||
| See [GitHub Pages documentation](https://help.github.com/articles/using-jekyll-with-pages/) for more details. | ||
|
|
||
| --- | ||
|
|
||
| ## Writing Guidelines | ||
|
|
||
| - Use clear, concise language suitable for a developer audience. | ||
| - Write content in **Markdown** (`.md` or `.markdown` files). | ||
| - Use relative links within the site where possible. | ||
| - When adding code examples, use fenced code blocks with a language tag: | ||
| ```` | ||
| ```sh | ||
| git log --oneline | ||
| ``` | ||
| ```` | ||
| Supported languages for syntax highlighting include: `sh`/`bash`, `c`, `diff`, `go`, `ini`, `python`, `ruby`, `yaml`, and others. | ||
| - Avoid adding unnecessary dependencies or large binary assets. | ||
|
|
||
| --- | ||
|
|
||
| ## Code Style | ||
|
|
||
| - **JavaScript / CSS:** Keep changes minimal and consistent with the existing style in `script/` and `css/`. | ||
| - **Jekyll Layouts & Plugins:** Changes to `_layouts/`, `_includes/`, or `_plugins/` should be well-tested locally before submitting. | ||
| - **Gemfile:** Do **not** modify the `Gemfile` unless there is a clear, necessary reason — it is shared and affects all contributors' environments. | ||
| - **Whitespace:** Prefer Unix line endings (LF). Avoid trailing whitespace. | ||
|
|
||
| --- | ||
|
|
||
| ## Git Rev News Newsletter | ||
|
|
||
| The [Git Rev News](https://git.github.io/rev_news/) newsletter is published monthly. Each edition is a Markdown file under `_posts/`. | ||
|
|
||
| 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)). | ||
|
|
||
|
Collaborator
There was a problem hiding this comment. Choose a reason for hiding this commentThe 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. |
||
| --- | ||
|
|
||
| ## Reporting Issues | ||
|
|
||
| Found a bug, broken link, or outdated content? Please [open an issue](https://github.com/git/git.github.io/issues/new) on GitHub with: | ||
|
|
||
| - A clear title describing the problem. | ||
| - The URL of the affected page (if applicable). | ||
| - Steps to reproduce or a description of what you expected vs. what you saw. | ||
|
|
||
| --- | ||
|
|
||
| We appreciate every contribution, big or small. Thank you for helping improve the Git community website! | ||
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.
Please avoid long lines like this. Instead please split them at around 72 characters.
Also please avoid
—characters. It looks too much AI generated.