forked from jeremyevans/sequel
-
Notifications
You must be signed in to change notification settings - Fork 0
/
Copy pathCONTRIBUTING
39 lines (30 loc) · 1.58 KB
/
CONTRIBUTING
1
2
3
4
5
6
7
8
9
10
11
12
13
14
15
16
17
18
19
20
21
22
23
24
25
26
27
28
29
30
31
32
33
34
35
36
37
38
39
Issue Guidelines
----------------
1) Issues should only be created for things that are definitely bugs.
If you are not sure that the behavior is a bug, ask about it on
IRC or the sequel-talk Google Group. GitHub Issues should not be
used as a help forum.
2) If you are sure it is a bug, then post a complete description of
the issue, the simplest possible self-contained example showing
the problem, the full backtrace of any exception, and for issues
involving database queries, an SQL log.
Pull Request Guidelines
-----------------------
1) Try to include tests for all new features and substantial bug
fixes. See the testing guide for details about testing Sequel.
2) Try to include documentation for all new features. In most cases
this should include RDoc method documentation, but updates to the
guides is also appropriate in some cases.
3) Follow the style conventions of the surrounding code. In most
cases, this is standard ruby style.
4) Do not submit whitespace changes with code changes. Sequel is not
pedantic about trailing whitespace, so if you have an editor that
automatically strips trailing whitespace, you may want to turn
that feature off.
5) All code in pull requests is assumed to be MIT licensed. Do not
submit a pull request if that isn't the case.
6) Sequel supports ruby 1.8.7+ and RSpec 1.3+. Please don't submit
pull requests that use ruby 1.9 hash syntax or other features not
supported by ruby 1.8.7, or specs that use features not supported
by RSpec 1.3. Please test on ruby 1.8.7 before submitting pull
requests.