| Summary: | [modeling] rename classes to follow standard naming convention | ||
|---|---|---|---|
| Product: | z_Archived | Reporter: | Steffen Pingel <steffen.pingel> |
| Component: | Mylyn | Assignee: | Miles Parker <milesparker> |
| Status: | RESOLVED FIXED | QA Contact: | |
| Severity: | enhancement | ||
| Priority: | P3 | ||
| Version: | unspecified | ||
| Target Milestone: | 0.9 | ||
| Hardware: | All | ||
| OS: | All | ||
| Whiteboard: | |||
| Bug Depends on: | |||
| Bug Blocks: | 355025 | ||
|
Description
Steffen Pingel
https://github.com/MilesParker/mylyn.incubator/commit/1c82a17b7ff5bc1f0257332185ef3845c1ac1894 FWVeryLittleIW, naturally modeling tools follow the opposite convention :) so that feels 'correct' to me, but it should have been obvious that Mylyn uses the other one. >"Mylyn" should not be used in class names, instead use identifiers that describe the purpose of a class, e.g. InterestDecoratorProvider (instead of MylynDecoratorProvider). https://github.com/MilesParker/mylyn.incubator/commit/47f35a4558865b8b760eaef90bb82829b1875237 Again, FWVLIW, here I was using Mylyn as a modifier to indicate the purpose of the class, which is to provide a generic IDecoratorProvider for all Mylyn implementations. But ContextDecoratorProvider describes it much better I think. |