[9.0][FIX] report_xls: Move description to first line #66
Conversation
| @@ -26,8 +26,7 @@ | |||
| 'author': "Noviat,Odoo Community Association (OCA)", | |||
| 'website': 'http://www.noviat.com', | |||
| 'category': 'Reporting', | |||
| 'description': """ | |||
| Excel report engine | |||
| 'description': """Excel report engine | |||
There was a problem hiding this comment.
Simply remove the description key. This is not mandatory anymore with the README file.
(even more this becomes redundant and difficult to keep in sync)
There was a problem hiding this comment.
In this instance there isn't a ReadMe, and I was trying to keep the change-set minimal as this isn't a full conversion & I am just trying to fix the build (all 9.0 PRs are failing right now). If this is a block however, I will move this over to a ReadMe using OCA template.
There was a problem hiding this comment.
You can cherry-commit fddca41 from 8.0
There was a problem hiding this comment.
Excellent call, thanks @pedrobaeza
There was a problem hiding this comment.
you could remove the description field as README.rst was added
lasley
force-pushed
the
hotfix/build-error
branch
from
4324c6c to
fa3c995
Compare
|
Oops forgot to drop a note saying comments handled after handling comments. Comments handled! 😄 |
I'm not sure why the RunBot builds are failing on a non-installable module, but they are and it's killing my OCD.
This is minimalistic change that
hopefullyfixes the builds.