https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=41521

--- Comment #30 from David Cook <[email protected]> ---
(In reply to Jonathan Druart from comment #29)
> (In reply to David Cook from comment #28)
> > Comment on attachment 192671 [details] [review] [review]
> > Bug 41521: Reject lib/ from tidy check
> > 
> > Review of attachment 192671 [details] [review] [review]:
> > -----------------------------------------------------------------
> > 
> > ::: Koha/Devel/Files.pm
> > @@ +35,4 @@
> > >              qw(
> > >                  Koha/Schema/Result
> > >                  Koha/Schema.pm
> > > +                lib/
> > 
> > At some point, I was hoping that we'd start moving our modules into this
> > directory though?
> 
> Yes, "at some point" :D

Haha fair. 

> We will adjust when we will be there I'd say. We will certainly need to move
> those to lib/External or lib/ThirdParty. But this is not a new pattern.

I'm not sure what you mean here?

> What do you suggest exactly?

I just mean rather than adding "lib/" to Koha/Devel/Files.pm, perhaps it would
be better just to add "lib/WebService/ILS/OverDrive/"?

-- 
You are receiving this mail because:
You are watching all bug changes.
_______________________________________________
Koha-bugs mailing list
[email protected]
https://lists.koha-community.org/cgi-bin/mailman/listinfo/koha-bugs
website : http://www.koha-community.org/
git : http://git.koha-community.org/
bugs : http://bugs.koha-community.org/

Reply via email to