Skip to content
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

Add option to set minimum supported branch #252

Merged
merged 1 commit into from
Jul 30, 2023

Conversation

sbidoul
Copy link
Member

@sbidoul sbidoul commented Jul 30, 2023

At some point we will not want to continue generating READMEs and publishing wheels for older Odoo versions.

@legalsylvain
Copy link
Collaborator

Naive question : why ?

@sbidoul
Copy link
Member Author

sbidoul commented Jul 30, 2023

The short answer is that we can't support old versions forever.

In more details

  • when I'll deploy the new README.rst generator, it will generate runboat links and runboat does not support Odoo 8 and 9 (it could but nobody contributed nor asked for it)
  • the oca/oca-ci image does not support Odoo 8 and 9 (it could but nobody contributed nor asked for it)
  • the OCA repo template does not support 8 and 9
  • sooner or later we'll stop paying for travis so we won't have CI anymore for 8 and 9, without official alternative since no one tests the repo template with 8 and 9

So I'm going to ask that we declare 8 and 9 officially unsupported by OCA.

@sbidoul
Copy link
Member Author

sbidoul commented Jul 30, 2023

Ah, I forgot to mention that the main branch bot consumes a significant amount of resources on the OCA server, so it will help with that too.

Nevertheless this PR does not change the default behaviour, it just adds a configuration option.

@sbidoul sbidoul force-pushed the main-branch-bot-min-version branch from e419036 to df58804 Compare July 30, 2023 20:22
@sbidoul sbidoul merged commit 43e6070 into OCA:master Jul 30, 2023
5 checks passed
@sbidoul sbidoul deleted the main-branch-bot-min-version branch July 30, 2023 20:33
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.

3 participants