-
-
Notifications
You must be signed in to change notification settings - Fork 479
[ADD] gen_addon_readme: text regarding OCA sponsoring program #468
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
base: master
Are you sure you want to change the base?
Conversation
6cdd830 to
0b42765
Compare
dreispt
left a comment
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
👍
0b42765 to
e438cda
Compare
|
The only problem on this is that it will be a lot of commits for updating whole OCA repos, saturating Travis/runbot, so this should be planned on advance. |
|
Hi @pedrobaeza . Thanks for your suggestion. I applied it.
We can just accept this PR, update OCA bot, and then, wait that natural PR (migration / patch) update the readme of modules. No need to update all the modules quickly, I think. Or did I miss something ? |
|
Can't these updates be incremental, as PRs are merged and individual READMES rebuilt with them? |
|
There's a cron in OCA server to update all READMEs of all repos, which is great, as you can synchronize any change. The only cons is that it should be planned as said. |
Thanks for the explanation. |
|
@pedrobaeza Does it make sense to drop the cron in favor of a precommit task? |
|
Well, it was done for avoiding manual interventions or rely on contributors for that synchronization, but you can tell @sbidoul that was the one that did it. |
|
Any chance to have this PR accepted ? IMO, it's a pity not to promote ways to support the OCA financially, on the OCA modules. |
|
@OCA/core-maintainers |
|
@legalsylvain Could you rebase ? |
Co-authored-by: Pedro M. Baeza <pedro.baeza@tecnativa.com>
df54642 to
6d2216a
Compare
|
rebased. |
|
Can we consider this is has been superseded by #656? |

add a simple text to promote sponsoring program of the OCA. Feel free to change / improve / fix typo editing directly the PR.
CC : @OCA/board