On Sun, Feb 9, 2014 at 11:16 PM, Thomas Mortagne
<thomas.mortagne(a)xwiki.com>wrote;wrote:
On Sun, Feb 9, 2014 at 6:10 PM, vincent(a)massol.net
<vincent(a)massol.net>
wrote:
Hi devs,
I always ask myself this question so I think we need a common agreement.
So here's the question:
* I have added some code in version N and this I have a "@since N" in
the code
* In version M (M > N), I move the
class/interface to a new package
Question: Do I change the @since annotation to "@since M" or not?
2 possibilities:
* Reasoning 1: it's a new class/interface since the FQN of the
class/interface
has changed and thus we should use "@since M"
* Reasoning 2: even though the FQN has changed
it's still the same code
that was moved and from a user POV, it was still
introduced in version N
and thus we should keep "@since N"
WDYT?
I'm hesitating. The most technically correct answer is Reasoning 1 IMO
but the
most useful one is probably Reasoning 2 since the question we wish
to answer is probably: "when was this code first introduced?".
Thus reasoning 2 seems slightly better to me.
Big -1 for 2 which is totally out of context, @since indicate that you
can use that class or method since that version in you code and
indicate you which version you are going to be compatible with. If you
change the class or method your can't keep the same @since. If you
want to know since when the feature exist look at
xwiki.org...
I completely agree with Thomas, a -1 for 2)
I would add that if you want to know from where the code come from, Git is
your best friend.
I take the occasion to also mention that it would be nice to have a better
way to maintain those @since. At least a check of presence, or even better
a check of correctness, in the build would nice to have. The must being to
have those @since added automagically :)
Thanks
-Vincent
_______________________________________________
devs mailing list
devs(a)xwiki.org
http://lists.xwiki.org/mailman/listinfo/devs
--
Thomas Mortagne
_______________________________________________
devs mailing list
devs(a)xwiki.org
http://lists.xwiki.org/mailman/listinfo/devs