On 28 Sep 2015 at 10:28:45, Thomas Mortagne ([email protected](mailto:[email protected])) wrote:
> I agree with this best practice at some point yes. > We need to agree on these 2 practices before we can implement it in AWM anyway :) Thanks -Vincent > On Mon, Sep 28, 2015 at 10:21 AM, [email protected] wrote: > > > > > > On 28 Sep 2015 at 10:06:52, Thomas Mortagne > > ([email protected](mailto:[email protected])) wrote: > > > >> Also before making this a rule we should probably refactor AWM first > >> since it's more and more the base for many new applications. > > > > Yes. > > > > I don’t think we need to wait to agree on what we want to have. The wait is > > only needed to decide when to apply the rule but we could decide the rule > > before applying it. > > > > Regarding AWM it’s planned for 7.3 (I talked about it with Marius a few > > days ago). I don’t know if we have a jira issue already but we should > > create one. > > > > Thanks > > -Vincent > > > > Ps: You haven’t said whether you agree with these rules (independently of > > when we would apply them) :) > > > >> On Mon, Sep 28, 2015 at 9:55 AM, [email protected] wrote: > >> > > >> > > >> > On 28 Sep 2015 at 09:25:51, Thomas Mortagne > >> > ([email protected](mailto:[email protected])) wrote: > >> > > >> >> Be careful with theses rules since they are not only for > >> >> xwiki-platform but also for contrib application and it should be made > >> >> clear that this is only for an application that require 7.2 or more > >> >> which is really not a good idea for a new contrib extension. > >> > > >> > Yes, good point, we should mention that this applies only when you > >> > depend on XWiki 7.2+. > >> > > >> > Note that this is already covered by this rule: > >> > > >> > "In your POM always try to depend on the oldest version of > >> > commons/rendering/platform dependencies for which your code works. At > >> > least ensure that your Applications works on the latest LTS version of > >> > XWiki. This will allow the largest number of users to use your > >> > application.” > >> > > >> > Thanks > >> > -Vincent > >> > > >> >> On Sun, Sep 27, 2015 at 9:19 PM, [email protected] wrote: > >> >> > Hi devs, > >> >> > > >> >> > Following our implementation of NS/NP in 7.2 I’d like to propose 2 > >> >> > new best practices for app dev that we would list at > >> >> > http://dev.xwiki.org/xwiki/bin/view/Community/ApplicationDevelopmentBestPractices > >> >> > > >> >> > 1) New rule 1: “Code” subspace > >> >> > > >> >> > Current text: > >> >> > * Generally, put all your pages in a single space dedicated for the > >> >> > application you're developing (e.g. Faq, Scheduler, IRC, > >> >> > AppWithinMinutes, etc). The name must be as short as possible while > >> >> > still being understandable of course and without overusing > >> >> > abbreviations. > >> >> > > >> >> > New version: > >> >> > * Generally, put all your pages in a single space dedicated for the > >> >> > application you're developing (e.g. Faq, Scheduler, IRC, > >> >> > AppWithinMinutes, etc). The name must be as short as possible while > >> >> > still being understandable of course and without overusing > >> >> > abbreviations. > >> >> > * Technical pages should be put in a subspace named “Code” > >> >> > > >> >> > Note: this rule can only be applied for new applications for now > >> >> > since the EM doesn’t know how to follow renames currently so for > >> >> > example if I move pages from the FAQCode space to the FAQ.Code space, > >> >> > when EM upgrades the app, it’ll display all pages in FAQCode as > >> >> > deleted (basically it considers all pages in FAQ.Code as new pages > >> >> > and pages in FAQCode as deleted pages). Note: I’ve created > >> >> > http://jira.xwiki.org/browse/XWIKI-12622 for this. > >> >> > > >> >> > 2) New rule 2: > >> >> > > >> >> > * Technical pages without children must be terminal pages. > >> >> > > >> >> > WDYT? > >> >> > > >> >> > Thanks > >> >> > -Vincent > >> >> > > >> >> > > >> >> > _______________________________________________ > >> >> > devs mailing list > >> >> > [email protected] > >> >> > http://lists.xwiki.org/mailman/listinfo/devs > >> >> > >> >> > >> >> > >> >> -- > >> >> Thomas Mortagne > >> >> _______________________________________________ > >> >> devs mailing list > >> >> [email protected] > >> >> http://lists.xwiki.org/mailman/listinfo/devs > >> > _______________________________________________ > >> > devs mailing list > >> > [email protected] > >> > http://lists.xwiki.org/mailman/listinfo/devs > >> > >> > >> > >> -- > >> Thomas Mortagne > >> _______________________________________________ > >> devs mailing list > >> [email protected] > >> http://lists.xwiki.org/mailman/listinfo/devs > > _______________________________________________ > > devs mailing list > > [email protected] > > http://lists.xwiki.org/mailman/listinfo/devs > > > > -- > Thomas Mortagne > _______________________________________________ > devs mailing list > [email protected] > http://lists.xwiki.org/mailman/listinfo/devs _______________________________________________ devs mailing list [email protected] http://lists.xwiki.org/mailman/listinfo/devs
