openerp-community team mailing list archive
Mailing list archive
Re: Convention for unported modules
As I stated on your PR, Daniel, these are the issues I see with _unported
"The only problem I see with this is that all modules will be listed when
you add the repository in the addons_path, that will dissapear when
migrations were done. This is even worst if someone forget to put
installable flag to False."
2014-07-01 15:13 GMT+02:00 Daniel Reis <dgreis@xxxxxxx>:
> Hi all,
> Current lp2git migration steps
> tell you to rename unported modules with a suffix "_unported".
> This was a strategy Sandy found to have TravisCI and Flake8 ignore them,
> and Guewen included that in the migration docs, and I believe it already
> been adopted in some of the migrations to GitHub.
> On the other hand, last November we also discussed here
> <https://lists.launchpad.net/openerp-community/msg04016.html> a
> convention to easily identify unported modules.
> I made the proposal to use a subdirectory, since I was working on a
> project where most modules were unported and often it wasn't easy to
> navigate around.
> We eventually agreed that a "__unported__" subdirectory could be used for
> I believe at the time most people didn't see that a necessity, so in
> practice it was not widely used.
> My problem is that the new convention does not solve the initial problem:
> uncluttering repo root from unported modules.
> And I believe the subdirectory approach is also capable to do what's
> needed for TravisCI and Flake8.
> So, my opinion is that we shoudl prefer the "__unported__" subdirectory
> convention over the "_unported" suffix convention.
> I believe the impact on already migrated repos is negligible.
> So I open the discussion on this, and of couse will follow whatever
> consensus comes out of it.
> Best regards,
> Daniel Reis
> Mailing list: https://launchpad.net/~openerp-community
> Post to : openerp-community@xxxxxxxxxxxxxxxxxxx
> Unsubscribe : https://launchpad.net/~openerp-community
> More help : https://help.launchpad.net/ListHelp