Skip to content

Migrate mail_environment to 10.0#662

Merged
lasley merged 2 commits into
OCA:10.0from
guewen:mig_10.0-mail_environment
Apr 6, 2017
Merged

Migrate mail_environment to 10.0#662
lasley merged 2 commits into
OCA:10.0from
guewen:mig_10.0-mail_environment

Conversation

@guewen

@guewen guewen commented Dec 21, 2016

Copy link
Copy Markdown
Member

need #595

@pedrobaeza pedrobaeza mentioned this pull request Dec 21, 2016
63 tasks
@guewen
guewen force-pushed the mig_10.0-mail_environment branch from d489e73 to e727653 Compare December 21, 2016 16:28
@yvaucher
yvaucher force-pushed the mig_10.0-mail_environment branch from e727653 to 0fada76 Compare March 23, 2017 12:57
@yvaucher

Copy link
Copy Markdown
Member

Tested and code LGTM

@moylop260

Copy link
Copy Markdown
Contributor

Could you fix mail_environment/README.rst:76: [E7901(rst-syntax-error), ] Bullet list ends without a blank line; unexpected unindent. and use 4 spaces for xml files, please?

@yvaucher

yvaucher commented Mar 24, 2017

Copy link
Copy Markdown
Member

@moylop260 this is a no for 4 spaces in xml files it was discussed long ago and people were mostly in favor of 2 spaces but for history reasons we didn't enforced it.

There are at least 554 xml files in OCA with 2 space indentation.

@gurneyalex
gurneyalex force-pushed the mig_10.0-mail_environment branch from 435f1fa to 53f9338 Compare April 6, 2017 14:18
@leemannd

leemannd commented Apr 6, 2017

Copy link
Copy Markdown
Contributor

@gurneyalex @guewen Could be merged?

@lasley
lasley merged commit 74dbf09 into OCA:10.0 Apr 6, 2017
SiesslPhillip pushed a commit to grueneerde/OCA-server-tools that referenced this pull request Nov 20, 2024
Syncing from upstream OCA/server-tools (14.0)
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

5 participants