| OLD | NEW |
| 1 <html> | 1 <html> |
| 2 <head> | 2 <head> |
| 3 <title>Contributing to FindBugs</title> | 3 <title>Contributing to FindBugs</title> |
| 4 <link rel="stylesheet" type="text/css" href="findbugs.css"> | 4 <link rel="stylesheet" type="text/css" href="findbugs.css"> |
| 5 | 5 |
| 6 </head> | 6 </head> |
| 7 <body> | 7 <body> |
| 8 | 8 |
| 9 <table width="100%"><tr> | 9 <table width="100%"><tr> |
| 10 | 10 |
| (...skipping 24 matching lines...) Expand all Loading... |
| 35 | 35 |
| 36 <tr><td><a class="sidebar" href="downloads.html"><b>Downloads</b></a></td></tr> | 36 <tr><td><a class="sidebar" href="downloads.html"><b>Downloads</b></a></td></tr> |
| 37 | 37 |
| 38 <tr><td> </td></tr> | 38 <tr><td> </td></tr> |
| 39 | 39 |
| 40 <tr><td><a class="sidebar" href="http://www.cafeshops.com/findbugs"><b>FindBugs
Swag</b></a></td></tr> | 40 <tr><td><a class="sidebar" href="http://www.cafeshops.com/findbugs"><b>FindBugs
Swag</b></a></td></tr> |
| 41 | 41 |
| 42 <tr><td> </td></tr> | 42 <tr><td> </td></tr> |
| 43 | 43 |
| 44 <tr><td><b>Development</b></td></tr> | 44 <tr><td><b>Development</b></td></tr> |
| 45 <tr><td><font size="-1"> <a class="sidebar" href="http://sourceforge.net/tr
acker/?group_id=96405">Open bugs</a></font></td></tr> | 45 <tr><td><font size="-1"> <a class="sidebar" href="http://sourceforge.net/p/
findbugs/bugs/">Open bugs</a></font></td></tr> |
| 46 <tr><td><font size="-1"> <a class="sidebar" href="reportingBugs.html">Repor
ting bugs</a></font></td></tr> | 46 <tr><td><font size="-1"> <a class="sidebar" href="reportingBugs.html">Repor
ting bugs</a></font></td></tr> |
| 47 <tr><td><font size="-1"> <a class="sidebar" href="contributing.html">Contri
buting</a></font></td></tr> | 47 <tr><td><font size="-1"> <a class="sidebar" href="contributing.html">Contri
buting</a></font></td></tr> |
| 48 <tr><td><font size="-1"> <a class="sidebar" href="team.html">Dev team</a></
font></td></tr> | 48 <tr><td><font size="-1"> <a class="sidebar" href="team.html">Dev team</a></
font></td></tr> |
| 49 <tr><td><font size="-1"> <a class="sidebar" href="api/index.html">API</a> <
a class="sidebar" href="api/overview-summary.html">[no frames]</a></font></td></
tr> | 49 <tr><td><font size="-1"> <a class="sidebar" href="api/index.html">API</a> <
a class="sidebar" href="api/overview-summary.html">[no frames]</a></font></td></
tr> |
| 50 <tr><td><font size="-1"> <a class="sidebar" href="Changes.html">Change log<
/a></font></td></tr> | 50 <tr><td><font size="-1"> <a class="sidebar" href="Changes.html">Change log<
/a></font></td></tr> |
| 51 <tr><td><font size="-1"> <a class="sidebar" href="http://sourceforge.net/pr
ojects/findbugs">SF project page</a></font></td></tr> | 51 <tr><td><font size="-1"> <a class="sidebar" href="http://sourceforge.net/pr
ojects/findbugs">SF project page</a></font></td></tr> |
| 52 <tr><td><font size="-1"> <a class="sidebar" href="http://code.google.com/p/
findbugs/source/browse/">Browse source</a></font></td></tr> | 52 <tr><td><font size="-1"> <a class="sidebar" href="http://code.google.com/p/
findbugs/source/browse/">Browse source</a></font></td></tr> |
| 53 <tr><td><font size="-1"> <a class="sidebar" href="http://code.google.com/p/
findbugs/source/list">Latest code changes</a></font></td></tr> | 53 <tr><td><font size="-1"> <a class="sidebar" href="http://code.google.com/p/
findbugs/source/list">Latest code changes</a></font></td></tr> |
| 54 </table> | 54 </table> |
| 55 </td> | 55 </td> |
| (...skipping 11 matching lines...) Expand all Loading... |
| 67 You can easily import FindBugs code into Eclipse by following the steps describe
d | 67 You can easily import FindBugs code into Eclipse by following the steps describe
d |
| 68 here: <a href="https://code.google.com/p/findbugs/source/browse/eclipsePlugin/do
c/building_findbugsplugin.txt">Import Eclipse projects</a> | 68 here: <a href="https://code.google.com/p/findbugs/source/browse/eclipsePlugin/do
c/building_findbugsplugin.txt">Import Eclipse projects</a> |
| 69 . | 69 . |
| 70 </p> | 70 </p> |
| 71 | 71 |
| 72 <h2>Preparing a patch</h2> | 72 <h2>Preparing a patch</h2> |
| 73 | 73 |
| 74 <p> The best way to create a patch is to clone FindBugs Git repository here: | 74 <p> The best way to create a patch is to clone FindBugs Git repository here: |
| 75 <a href="http://code.google.com/p/findbugs/source/clones">http://code.google.co
m/p/findbugs/source/clones</a>, | 75 <a href="http://code.google.com/p/findbugs/source/clones">http://code.google.co
m/p/findbugs/source/clones</a>, |
| 76 commit changes with descriptive commit messages (small commits are better) and | 76 commit changes with descriptive commit messages (small commits are better) and |
| 77 create a new <a href="https://sourceforge.net/p/findbugs/patches/">patch ticket
</a> | 77 create a new <a href="https://sourceforge.net/p/findbugs/bugs/">bug</a> |
| 78 with the link to your repository. | 78 or <a href="https://sourceforge.net/p/findbugs/feature-requests/">feature reque
st</a> |
| 79 with the "[patch]" prefix in the title and link to your repository in the descr
iption. |
| 79 </p> | 80 </p> |
| 80 | 81 |
| 81 <p> Please follow these guidelines when preparing your patch:</p> | 82 <p> Please follow these guidelines when preparing your patch:</p> |
| 82 <ul> | 83 <ul> |
| 83 <li> <b>Use the same indentation style as the source file(s) you | 84 <li> <b>Use the same indentation style as the source file(s) you |
| 84 are modifying</b>. In particular, please use spaces | 85 are modifying</b>. In particular, please use spaces |
| 85 to indent your code; four spaces per indent level. | 86 to indent your code; four spaces per indent level. |
| 86 <li> If at all possible, avoid making whitespace modifications. | 87 <li> If at all possible, avoid making whitespace modifications. |
| 87 <li> Small patches/changesets are appreciated. | 88 <li> Small patches/changesets are appreciated. |
| 89 <li> If you have lot of changes, try to group them by small commits with descrip
tive messages. |
| 90 <li> If you have multiple patches, create <b>branch per patch</b>. |
| 91 <li> All new files should contain proper license header (<a href="http://www.gnu
.org/licenses/lgpl.html">Lesser GNU Public License</a>). |
| 88 <li> If you are submitting a new bug detector, please submit a small | 92 <li> If you are submitting a new bug detector, please submit a small |
| 89 standalone source file that contains an instance of the | 93 standalone source file that contains an instance of the |
| 90 kind of bug the detector looks for. | 94 kind of bug the detector looks for. The file should be placed at |
| 95 <a href="https://code.google.com/p/findbugs/source/browse/#git%2FfindbugsTe
stCases%2Fsrc%2Fjava%2FsfBugsNew">findbugsTestCases/src/java/sfBugsNew</a> |
| 96 package and named after the bug or feature you are addressing. |
| 91 </ul> | 97 </ul> |
| 92 | 98 |
| 93 <p> Following these guidelines makes it much easier for us | 99 <p> Following these guidelines makes it much easier for us |
| 94 to incorporate new code. | 100 to incorporate new code. |
| 95 | 101 |
| 96 <h2>How to submit a patch</h2> | 102 <h2>Where to submit a patch</h2> |
| 97 | 103 |
| 98 <p> Patches may be submitted through the | 104 <p> Patches may be submitted through the |
| 99 <a href="https://sourceforge.net/p/findbugs/patches/">Patches</a> tracker on the | 105 <a href="https://sourceforge.net/p/findbugs/bugs/">bug</a> |
| 106 or <a href="https://sourceforge.net/p/findbugs/feature-requests/">feature reque
st</a> trackers on the |
| 100 <a href="http://sourceforge.net/projects/findbugs/">sourceforge project page</a>
. | 107 <a href="http://sourceforge.net/projects/findbugs/">sourceforge project page</a>
. |
| 101 | 108 |
| 102 | 109 |
| 103 <hr> <p> | 110 <hr> <p> |
| 104 <script language="JavaScript" type="text/javascript"> | 111 <script language="JavaScript" type="text/javascript"> |
| 105 <!---//hide script from old browsers | 112 <!---//hide script from old browsers |
| 106 document.write( "Last updated "+ document.lastModified + "." ); | 113 document.write( "Last updated "+ document.lastModified + "." ); |
| 107 //end hiding contents ---> | 114 //end hiding contents ---> |
| 108 </script> | 115 </script> |
| 109 <p> Send comments to <a class="sidebar" href="mailto:findbugs@cs.umd.edu">findbu
gs@cs.umd.edu</a> | 116 <p> Send comments to <a class="sidebar" href="mailto:findbugs@cs.umd.edu">findbu
gs@cs.umd.edu</a> |
| 110 <p> | 117 <p> |
| 111 <A href="http://sourceforge.net"><IMG src="http://sourceforge.net/sflogo.php?gro
up_id=96405&type=5" width="210" height="62" border="0" alt="SourceForge.net
Logo" /></A> | 118 <a href="http://sourceforge.net/projects/findbugs/"><img src="http://sourceforge
.net/sflogo.php?group_id=96405&type=5" width="210" height="62" border="0" al
t="FindBugs on SourceForge.net" /></a> |
| 112 | 119 |
| 113 </td> | 120 </td> |
| 114 | 121 |
| 115 </tr></table> | 122 </tr></table> |
| 116 | 123 |
| 117 </body> | 124 </body> |
| 118 </html> | 125 </html> |
| OLD | NEW |