Update README.rst #301
Update README.rst #301
Conversation
Add link to API Add license badge Format a little
README.rst
Outdated
| .. image:: https://img.shields.io/badge/License-AGPL%20v3-blue.svg | ||
| :target: https://www.gnu.org/licenses/agpl-3.0 | ||
| :alt: License: AGPL v3 |
There was a problem hiding this comment.
| .. image:: https://img.shields.io/badge/License-AGPL%20v3-blue.svg | |
| :target: https://www.gnu.org/licenses/agpl-3.0 | |
| :alt: License: AGPL v3 | |
| .. image:: https://img.shields.io/badge/license-AGPL--3-blue.png | |
| :target: https://www.gnu.org/licenses/agpl-3.0 | |
| :alt: License: AGPL-3 |
There was a problem hiding this comment.
Lol, I literally cut and paste that from a recommended gist - no problems.
There was a problem hiding this comment.
I think I should revert the image, that png is dog ugly compared to the nice clean svg. Is there any reason we want png specifically?
There was a problem hiding this comment.
Oh, as you wish. Just FYI svg files are not rendered in apps, thus I always put .png in readme files by default.
There was a problem hiding this comment.
Good enough reason for me, although the other images are svg's, I'll leave it.
There was a problem hiding this comment.
There was a problem hiding this comment.
Look the date of such change... I think this must be re-evaluated.
There was a problem hiding this comment.
Well in any case, this is not an app.
Should I maybe convert to Markdown too? RST was a bit funky to edit remembering the slight syntax differences.
There was a problem hiding this comment.
Yeah, this is not an app. Let's put svg then!
There was a problem hiding this comment.
And we need to review the criteria for the rest, as Odoo has evolved. First of all, we nee to know where the incompatibility was.
Per Review Co-authored-by: Miquel Raïch <miquel.raich@forgeflow.com>
|
@gdgellatly : could you rebase and fix conflict ? |
|
/ocabot rebase |
|
@gdgellatly The rebase process failed, because command |
|
@gdgellatly Could you give one last push to this PR? needs rebase (to solve conflicts) and squash commits... |
|
/ocabot rebase |
|
@gdgellatly The rebase process failed, because command |
|
|
||
| pip/pip3 install --ignore-installed git+https://github.com/OCA/openupgradelib.git@master | ||
| ``pip install --ignore-installed git+https://github.com/OCA/openupgradelib.git@master`` |
There was a problem hiding this comment.
Now that the release to Pypi process is automated, the library is released regularly and there is no need to install from Github anymore.
There was a problem hiding this comment.
lol ffs, try and do something little 3 years ago. Will fix Wednesday.
Add link to API
Add license badge
Format a little