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] full CI #13

Merged
merged 17 commits into from
Aug 1, 2023
Merged

[ADD] full CI #13

merged 17 commits into from
Aug 1, 2023

Conversation

hbrunn
Copy link
Contributor

@hbrunn hbrunn commented Jul 31, 2023

@albig da ich doch noch diverse Dinge korrigieren musste, hier ein separater PR, hier können wir vielleicht auch noch ein paar technische Punkte besprechen.

Die Magie passiert in https://github.com/verdigado/odoo-customize/blob/15.0/.github/workflows/test.yml#L56 wo ich die requirements installiere und auf Basis der repo.yml des Projekts die anderen OCA-Repositories pulle. Der Rest ist dann das in https://github.com/OCA/oca-ci einzupassen, nicht spannend für unsere Zwecke.

Coverage finde ich noch ein hilfreiches Feature, das produziert Übersichten wie https://app.codecov.io/gh/OCA/server-backend/pull/231 mit denen Du verifizieren kannst dass die Tests allen relevanten Code durchlaufen. Das braucht aber eine einmalige (kostenlose weil öffentliches Projekt) Registrierung des repos durch einen repo-admin bei https://codecov.io.

In https://github.com/verdigado/odoo-customize/blob/15.0/.github/workflows/test.yml#L85 gibt es dann noch einen automatischen Export+checkin der po-Files, das habe ich von einem anderen Kunden übernommen bei dem ich das sehr nützlich finde. Auf die Weise müssen sich devs nicht um Übersetzungen kümmern, und ein Consultant kann sie (auch per github-UI ohne git zu verstehen) bearbeiten. Das habe ich aber soeben deaktiviert weil das ohne Änderungen nur funktioniert wenn ich auf einen Branch in diesem Repo pushe, nicht auf meinen eigenen Fork.

@hbrunn hbrunn marked this pull request as ready for review July 31, 2023 16:49
@hbrunn hbrunn closed this Jul 31, 2023
@hbrunn hbrunn reopened this Jul 31, 2023
@albig albig merged commit a43feaf into verdigado:15.0 Aug 1, 2023
4 checks passed
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.

2 participants