-
-
Notifications
You must be signed in to change notification settings - Fork 706
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
[REF] Remove docs from 16.0 branch #3853
Conversation
CC :
Note : you can see the altered readme here : https://github.com/legalsylvain/OpenUpgrade/tree/16.0-REM-documentation |
I'm seeing one problem: any merge in 16.0 branch won't trigger a new documentation build. That was one of the reasons for not splitting the documentation branch. |
Hi. Thanks for your reviews !
Yes, the problem was raised in the original topic and I answered here : A PoC is available, feel free to take a look on it. I'll do a PR against the "master" branch. (16.0 for the time being) to add this file : Note : that is unrelated to that PR. For now, the github workflow present in the 16.0 branch and removed by this PR doesn't deploy on oca.github.io/openupgrade |
Note : i checked again github doc, and instead of using cron, we can use path filter. Pseudo workflow : Pull : This will even save more CI ! Anyway, @pedrobaeza, unrelated to this PR. Is this one OK for you ? |
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.
Thanks for the cleanup
remove all docs folder
remove all docsource folder, except coverage file of the current version. (15 -> 16)
simplify pre-commit stuff, removing exception regarding docs / docsource folders.
minor changes in main README.md file
Consequency :
Note : once validated, I'll do the same for recent branches with new modules design. (14 / 15)