1
1
# Contributing to senaite.core
2
2
3
- Third-party contributions are essential for keeping ` senaite.core ` continuously
4
- improving. We simply cannot access the huge number of platforms and myriad
5
- configurations for running ` senaite.core ` . We want to keep it as easy as
6
- possible to contribute changes that get things working in your environment.
7
- There are a few guidelines that we need contributors to follow so that we can
3
+ Third-party contributions are essential for keeping ` senaite.core ` continuously
4
+ improving. We simply cannot access the huge number of platforms and myriad
5
+ configurations for running ` senaite.core ` . We want to keep it as easy as
6
+ possible to contribute changes that get things working in your environment.
7
+ There are a few guidelines that we need contributors to follow so that we can
8
8
have a chance of keeping on top of things.
9
9
10
- The following is a set of guidelines for contributing to senaite.core, which is
11
- hosted in the [ SENAITE Organization] ( https://github.com/senaite ) on GitHub.
12
- These are just guidelines, not rules. Use your best judgment, and feel free to
10
+ The following is a set of guidelines for contributing to senaite.core, which is
11
+ hosted in the [ SENAITE Organization] ( https://github.com/senaite ) on GitHub.
12
+ These are just guidelines, not rules. Use your best judgment, and feel free to
13
13
propose changes to this document in a [ pull request] ( #how-to-submit-a-pull-request ) .
14
14
15
15
## Code of Conduct
@@ -63,21 +63,21 @@ contribute to the source code.
63
63
https://github.com/senaite/senaite.core/fork ). This will copy the whole
64
64
` senaite.core ` repository to your personal account.
65
65
66
- 3 . Then, go to your favourite working folder in your computer and clone your
66
+ 3 . Then, go to your favourite working folder in your computer and clone your
67
67
forked repository by typing (replacing ``` YOUR_USERNAME ``` by
68
68
the actual username of your GitHub account):
69
69
70
70
$ git clone https://github.com/YOUR_USERNAME/senaite.core
71
71
72
- 4 . Your forked repository https://github.com/YOUR_USERNAME/senaite.core will
73
- receive the default name of ` origin ` . You can also add the original
72
+ 4 . Your forked repository https://github.com/YOUR_USERNAME/senaite.core will
73
+ receive the default name of ` origin ` . You can also add the original
74
74
` senaite.core ` repository, which is usually called ` upstream ` :
75
75
76
76
$ cd senaite.core
77
77
$ git remote add upstream https://github.com/senaite/senaite.core.git
78
78
79
- To verify the new upstream repository you have specified for your fork, type
80
- ` git remote -v ` . You should see the URL for your fork as ` origin ` , and the URL
79
+ To verify the new upstream repository you have specified for your fork, type
80
+ ` git remote -v ` . You should see the URL for your fork as ` origin ` , and the URL
81
81
for the original repository as ` upstream ` :
82
82
83
83
```
@@ -99,15 +99,15 @@ $ git pull upstream master
99
99
```
100
100
101
101
Now you can do changes, add files, do commits (please take a look at
102
- [ how to write good commit messages] ( https://chris.beams.io/posts/git-commit/ ) !)
102
+ [ how to write good commit messages] ( https://chris.beams.io/posts/git-commit/ ) !)
103
103
and push them to your repository:
104
104
105
105
```
106
106
$ git push origin my_feature
107
107
```
108
108
109
109
If there have been new pushes to the ` master ` branch of the ` upstream `
110
- repository since the last time you pulled from it, you might want to put your
110
+ repository since the last time you pulled from it, you might want to put your
111
111
commits on top of them (this is mandatory for pull requests):
112
112
113
113
```
@@ -129,7 +129,7 @@ pros and cons between doing a `gir rebase` or `git merge`. Good documentation on
129
129
this regard can be found in the [ Atlassian's Merging vs. Rebasing tutorial] (
130
130
https://www.atlassian.com/git/tutorials/merging-vs-rebasing ).
131
131
132
- Note this ` git rebase ` or ` git merge ` is required for keeping your branch
132
+ Note this ` git rebase ` or ` git merge ` is required for keeping your branch
133
133
aligned with the latest code from the repos. The incorporation of your work into
134
134
` master ` through a Pull Request will always be done using ` git merge ` .
135
135
@@ -141,15 +141,15 @@ GitHub repository, switch to your `my_feature` branch, and click the
141
141
_ ** Pull Request** _ button, which will do all the work for you. Ensure the
142
142
comparison is done with the ` master ` branch unless you forked from another one.
143
143
144
- Once a pull request is sent, the Developer Team will review the set of changes,
144
+ Once a pull request is sent, the Developer Team will review the set of changes,
145
145
discuss potential modifications, and even push follow-up commits if necessary.
146
146
147
147
Some things that will increase the chance that your pull request is accepted:
148
148
149
149
* Write tests.
150
150
* Follow [ Plone's Python styleguide] ( https://docs.plone.org/develop/styleguide/python.html ) .
151
151
* Write a descriptive and detailed summary. Please consider that reviewing pull
152
- requests is hard, so include as much information as possible to make your
152
+ requests is hard, so include as much information as possible to make your
153
153
pull request's intent clear.
154
154
* Do not address multiple bugfixes or features in the same Pull Request.
155
155
* Include whitespace and formatting changes in discrete commits.
@@ -161,17 +161,17 @@ For more details about Git usage, please check out Chapters 1 and 2 from
161
161
162
162
## Contributing with new ideas
163
163
164
- All suggestions and proposals are welcome. We strongly believe that the
165
- feedback of the community is an important asset to make a better project. With
166
- the aim to get the most of these contributions, but without interfering with
167
- the undergoing work regarding to issues and Pull Requests, we've created a
164
+ All suggestions and proposals are welcome. We strongly believe that the
165
+ feedback of the community is an important asset to make a better project. With
166
+ the aim to get the most of these contributions, but without interfering with
167
+ the undergoing work regarding to issues and Pull Requests, we've created a
168
168
[ Community discussion board] (
169
- https://community.senaite.org ). This is the right place if
169
+ https://community.senaite.org ). This is the right place if
170
170
you are willing to discuss about new ideas, further steps or improvements.
171
171
172
- If you want to keep in touch with the community members and up-to-date
172
+ If you want to keep in touch with the community members and up-to-date
173
173
with the latest discussions, please join to the [ Gitter community channel] (
174
174
https://gitter.im/senaite/Lobby ).
175
175
176
- If you want to stay informed about senaite at easy pace, don't forget
176
+ If you want to stay informed about senaite at easy pace, don't forget
177
177
to subscribe to our [ users list] ( https://sourceforge.net/projects/senaite/lists/senaite-users )
0 commit comments