mirror of
				https://github.com/ohmyzsh/ohmyzsh.git
				synced 2024-05-11 05:55:17 +00:00 
			
		
		
		
	
		
			
				
	
	
		
			225 lines
		
	
	
		
			8.1 KiB
		
	
	
	
		
			Markdown
		
	
	
	
	
	
			
		
		
	
	
			225 lines
		
	
	
		
			8.1 KiB
		
	
	
	
		
			Markdown
		
	
	
	
	
	
# CONTRIBUTING GUIDELINES
 | 
						|
 | 
						|
Oh-My-Zsh is a community-driven project. Contribution is welcome, encouraged, and appreciated.
 | 
						|
It is also essential for the development of the project.
 | 
						|
 | 
						|
First, please take a moment to review our [code of conduct](CODE_OF_CONDUCT.md).
 | 
						|
 | 
						|
These guidelines are an attempt at better addressing the huge amount of pending
 | 
						|
issues and pull requests. Please read them closely.
 | 
						|
 | 
						|
Foremost, be so kind as to [search](#use-the-search-luke). This ensures any contribution
 | 
						|
you would make is not already covered.
 | 
						|
 | 
						|
<!-- TOC updateonsave:true depthfrom:2 -->
 | 
						|
 | 
						|
- [Reporting Issues](#reporting-issues)
 | 
						|
  - [You have a problem](#you-have-a-problem)
 | 
						|
  - [You have a suggestion](#you-have-a-suggestion)
 | 
						|
- [Submitting Pull Requests](#submitting-pull-requests)
 | 
						|
  - [Getting started](#getting-started)
 | 
						|
  - [You have a solution](#you-have-a-solution)
 | 
						|
  - [You have an addition](#you-have-an-addition)
 | 
						|
- [Use the Search, Luke](#use-the-search-luke)
 | 
						|
- [Commit Guidelines](#commit-guidelines)
 | 
						|
  - [Format](#format)
 | 
						|
  - [Style](#style)
 | 
						|
- [Volunteer](#volunteer)
 | 
						|
 | 
						|
<!-- /TOC -->
 | 
						|
 | 
						|
## Reporting Issues
 | 
						|
 | 
						|
### You have a problem
 | 
						|
 | 
						|
Please be so kind as to [search](#use-the-search-luke) for any open issue already covering
 | 
						|
your problem.
 | 
						|
 | 
						|
If you find one, comment on it so we can know there are more people experiencing it.
 | 
						|
 | 
						|
If not, look at the [Troubleshooting](https://github.com/ohmyzsh/ohmyzsh/wiki/Troubleshooting)
 | 
						|
page for instructions on how to gather data to better debug your problem.
 | 
						|
 | 
						|
Then, you can go ahead and create an issue with as much detail as you can provide.
 | 
						|
It should include the data gathered as indicated above, along with:
 | 
						|
 | 
						|
1. How to reproduce the problem
 | 
						|
2. What the correct behavior should be
 | 
						|
3. What the actual behavior is
 | 
						|
 | 
						|
Please copy to anyone relevant (_eg_ plugin maintainers) by mentioning their GitHub handle
 | 
						|
(starting with `@`) in your message.
 | 
						|
 | 
						|
We will do our very best to help you.
 | 
						|
 | 
						|
### You have a suggestion
 | 
						|
 | 
						|
Please be so kind as to [search](#use-the-search-luke) for any open issue already covering
 | 
						|
your suggestion.
 | 
						|
 | 
						|
If you find one, comment on it so we can know there are more people supporting it.
 | 
						|
 | 
						|
If not, you can go ahead and create an issue. Please copy to anyone relevant (_eg_ plugin
 | 
						|
maintainers) by mentioning their GitHub handle (starting with `@`) in your message.
 | 
						|
 | 
						|
## Submitting Pull Requests
 | 
						|
 | 
						|
### Getting started
 | 
						|
 | 
						|
You should be familiar with the basics of
 | 
						|
[contributing on GitHub](https://help.github.com/articles/using-pull-requests) and have a fork
 | 
						|
[properly set up](https://github.com/ohmyzsh/ohmyzsh/wiki/Contribution-Technical-Practices).
 | 
						|
 | 
						|
You MUST always create PRs with _a dedicated branch_ based on the latest upstream tree.
 | 
						|
 | 
						|
If you create your own PR, please make sure you do it right. Also be so kind as to reference
 | 
						|
any issue that would be solved in the PR description body,
 | 
						|
[for instance](https://help.github.com/articles/closing-issues-via-commit-messages/)
 | 
						|
_"Fixes #XXXX"_ for issue number XXXX.
 | 
						|
 | 
						|
### You have a solution
 | 
						|
 | 
						|
Please be so kind as to [search](#use-the-search-luke) for any open issue already covering
 | 
						|
your [problem](#you-have-a-problem), and any pending/merged/rejected PR covering your solution.
 | 
						|
 | 
						|
If the solution is already reported, try it out and +1 the pull request if the
 | 
						|
solution works ok. On the other hand, if you think your solution is better, post
 | 
						|
it with a reference to the other one so we can have both solutions to compare.
 | 
						|
 | 
						|
If not, then go ahead and submit a PR. Please copy to anyone relevant (e.g. plugin
 | 
						|
maintainers) by mentioning their GitHub handle (starting with `@`) in your message.
 | 
						|
 | 
						|
### You have an addition
 | 
						|
 | 
						|
Please [do not](https://github.com/ohmyzsh/ohmyzsh/wiki/Themes#dont-send-us-your-theme-for-now)
 | 
						|
send themes for now.
 | 
						|
 | 
						|
Please be so kind as to [search](#use-the-search-luke) for any pending, merged or rejected Pull Requests
 | 
						|
covering or related to what you want to add.
 | 
						|
 | 
						|
If you find one, try it out and work with the author on a common solution.
 | 
						|
 | 
						|
If not, then go ahead and submit a PR. Please copy to anyone relevant (_eg_ plugin
 | 
						|
maintainers) by mentioning their GitHub handle (starting with `@`) in your message.
 | 
						|
 | 
						|
For any extensive change, _eg_ a new plugin, you will have to find testers to +1 your PR.
 | 
						|
 | 
						|
----
 | 
						|
 | 
						|
## Use the Search, Luke
 | 
						|
 | 
						|
_May the Force (of past experiences) be with you_
 | 
						|
 | 
						|
GitHub offers [many search features](https://help.github.com/articles/searching-github/)
 | 
						|
to help you check whether a similar contribution to yours already exists. Please search
 | 
						|
before making any contribution, it avoids duplicates and eases maintenance. Trust me,
 | 
						|
that works 90% of the time.
 | 
						|
 | 
						|
You can also take a look at the [FAQ](https://github.com/ohmyzsh/ohmyzsh/wiki/FAQ)
 | 
						|
to be sure your contribution has not already come up.
 | 
						|
 | 
						|
If all fails, your thing has probably not been reported yet, so you can go ahead
 | 
						|
and [create an issue](#reporting-issues) or [submit a PR](#submitting-pull-requests).
 | 
						|
 | 
						|
----
 | 
						|
 | 
						|
## Commit Guidelines
 | 
						|
 | 
						|
Oh My Zsh uses the [Conventional Commits](https://www.conventionalcommits.org/en/v1.0.0/)
 | 
						|
specification. The automatic changelog tool uses these to automatically generate
 | 
						|
a changelog based on the commit messages. Here's a guide to writing a commit message
 | 
						|
to allow this:
 | 
						|
 | 
						|
### Format
 | 
						|
 | 
						|
```
 | 
						|
type(scope)!: subject
 | 
						|
```
 | 
						|
 | 
						|
- `type`: the type of the commit is one of the following:
 | 
						|
 | 
						|
  - `feat`: new features.
 | 
						|
  - `fix`: bug fixes.
 | 
						|
  - `docs`: documentation changes.
 | 
						|
  - `refactor`: refactor of a particular code section without introducing
 | 
						|
    new features or bug fixes.
 | 
						|
  - `style`: code style improvements.
 | 
						|
  - `perf`: performance improvements.
 | 
						|
  - `test`: changes to the test suite.
 | 
						|
  - `ci`: changes to the CI system.
 | 
						|
  - `build`: changes to the build system (we don't yet have one so this shouldn't apply).
 | 
						|
  - `chore`: for other changes that don't match previous types. This doesn't appear
 | 
						|
    in the changelog.
 | 
						|
 | 
						|
- `scope`: section of the codebase that the commit makes changes to. If it makes changes to
 | 
						|
  many sections, or if no section in particular is modified, leave blank without the parentheses.
 | 
						|
  Examples:
 | 
						|
 | 
						|
  - Commit that changes the `git` plugin:
 | 
						|
  ```
 | 
						|
  feat(git): add alias for `git commit`
 | 
						|
  ```
 | 
						|
 | 
						|
  - Commit that changes many plugins:
 | 
						|
  ```
 | 
						|
  style: fix inline declaration of arrays
 | 
						|
  ```
 | 
						|
 | 
						|
  For changes to plugins or themes, the scope should be the plugin or theme name:
 | 
						|
 | 
						|
  - ✅ `fix(agnoster): commit subject`
 | 
						|
  - ❌ `fix(theme/agnoster): commit subject`
 | 
						|
 | 
						|
- `!`: this goes after the `scope` (or the `type` if scope is empty), to indicate that the commit
 | 
						|
  introduces breaking changes.
 | 
						|
 | 
						|
  Optionally, you can specify a message that the changelog tool will display to the user to indicate
 | 
						|
  what's changed and what they can do to deal with it. You can use multiple lines to type this message;
 | 
						|
  the changelog parser will keep reading until the end of the commit message or until it finds an empty
 | 
						|
  line.
 | 
						|
 | 
						|
  Example (made up):
 | 
						|
 | 
						|
  ```
 | 
						|
  style(agnoster)!: change dirty git repo glyph
 | 
						|
 | 
						|
  BREAKING CHANGE: the glyph to indicate when a git repository is dirty has
 | 
						|
  changed from a Powerline character to a standard UTF-8 emoji. You can
 | 
						|
  change it back by setting `ZSH_THEME_DIRTY_GLYPH`.
 | 
						|
 | 
						|
  Fixes #420
 | 
						|
 | 
						|
  Co-authored-by: Username <email>
 | 
						|
  ```
 | 
						|
 | 
						|
- `subject`: a brief description of the changes. This will be displayed in the changelog. If you need
 | 
						|
  to specify other details you can use the commit body but it won't be visible.
 | 
						|
 | 
						|
  Formatting tricks: the commit subject may contain:
 | 
						|
 | 
						|
  - Links to related issues or PRs by writing `#issue`. This will be highlighted by the changelog tool:
 | 
						|
    ```
 | 
						|
    feat(archlinux): add support for aura AUR helper (#9467)
 | 
						|
    ```
 | 
						|
 | 
						|
  - Formatted inline code by using backticks: the text between backticks will also be highlighted by
 | 
						|
    the changelog tool:
 | 
						|
    ```
 | 
						|
    feat(shell-proxy): enable unexported `DEFAULT_PROXY` setting (#9774)
 | 
						|
    ```
 | 
						|
 | 
						|
### Style
 | 
						|
 | 
						|
Try to keep the first commit line short. This is harder to do using this commit style but try to be
 | 
						|
concise and if you need more space, you can use the commit body. Try to make sure that the commit
 | 
						|
subject is clear and precise enough that users will know what change by just looking at the changelog.
 | 
						|
 | 
						|
----
 | 
						|
 | 
						|
## Volunteer
 | 
						|
 | 
						|
Very nice!! :)
 | 
						|
 | 
						|
Please have a look at the [Volunteer](https://github.com/ohmyzsh/ohmyzsh/wiki/Volunteers)
 | 
						|
page for instructions on where to start and more.
 |