Summary: | [Wizards] new extension points for commit operation | ||||||
---|---|---|---|---|---|---|---|
Product: | [Eclipse Project] Platform | Reporter: | Christian Plaetzinger <christian.plaetzinger> | ||||
Component: | CVS | Assignee: | platform-cvs-inbox <platform-cvs-inbox> | ||||
Status: | ASSIGNED --- | QA Contact: | |||||
Severity: | enhancement | ||||||
Priority: | P5 | CC: | brockj, mik.kersten | ||||
Version: | 3.2.1 | Keywords: | contributed, helpwanted | ||||
Target Milestone: | --- | ||||||
Hardware: | All | ||||||
OS: | All | ||||||
Whiteboard: | |||||||
Attachments: |
|
Description
Christian Plaetzinger
2007-01-12 04:55:12 EST
Created attachment 56817 [details]
Changed and new resources
Thanks for the contribution. Here are some comments: 1) Could you reattach the contribution in the form of a patch? That way it will be easier to ensure that changes made to HEAD since you made changes will not be lost (and also makes it easier to browse the patch to inspect changes). 2) Any new files require a copyright. The easiest approach is to take the file header from an existing file and change the contributing company as appropriate. You will also want to add a contribution line to any modified files. 3) Could you add documentation to the scheme files describing the purpose of the extension and how a client should use it. 4) Eclipse convention is to start interfaces with I (e.g. ICommitCommentContributor). Due to comment 1, I haven't done a thorough inspection of the code. I will perform a more in depth inspection once the contribution is in patch form. This bug hasn't had any activity in quite some time. Maybe the problem got resolved, was a duplicate of something else, or became less pressing for some reason - or maybe it's still relevant but just hasn't been looked at yet. If you have further information on the current state of the bug, please add it. The information can be, for example, that the problem still occurs, that you still want the feature, that more information is needed, or that the bug is (for whatever reason) no longer relevant. |