You signed in with another tab or window. Reload to refresh your session.You signed out in another tab or window. Reload to refresh your session.You switched accounts on another tab or window. Reload to refresh your session.Dismiss alert
{{ message }}
Repository navigation
Commit 79239a4
Browse filesBrowse the repository at this point in the historyBrowse files
6. When at all possible, code contributions should include tests. These can
134
175
either be added to an existing test, or completely new. Please see
135
176
[test/README.md](test/README.md) for information on the test framework.
@@ -152,3 +193,35 @@ guidelines:
152
193
153
194
9. Guidelines on how to integrate error output of new crypto library modules
154
195
can be found in [crypto/err/README.md](crypto/err/README.md).
196
+
197
+
10. Once your Pull Request gets to the stage of being reviewed fixup commits
198
+
should be used where possible. Fixup commits are squashed when the PR is
199
+
finally merged. Fixup commits are done in the following way:
200
+
201
+
```
202
+
203
+
# Add one or more updated files that needed changes
204
+
git add <filename>
205
+
206
+
# Do a fixup commit
207
+
# <commit-id> is the id of a previous commit that you want to fix up.
208
+
git commit --fixup <commit-id>
209
+
210
+
# Do a non forced push
211
+
git push
212
+
```
213
+
214
+
To view commit-id's use:
215
+
216
+
```
217
+
git log
218
+
```
219
+
220
+
11. If a Pull Request addresses an [issue](https://github.com/openssl/openssl/issues/)
221
+
the commit should include the line:
222
+
223
+
```
224
+
Fixes: LINK
225
+
```
226
+
227
+
where LINK is the https link to the issue in github.
0 commit comments