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

Implemented proper Docker tags #2217

Merged
merged 4 commits into from Jan 6, 2022
Merged

Implemented proper Docker tags #2217

merged 4 commits into from Jan 6, 2022

Conversation

goebelmeier
Copy link
Contributor

Changed latest tag to nightly for nightly builds
Added 0 tag to current release builds
Fixes #2216

chrissooo and others added 3 commits January 6, 2022 07:50
HIer wurde ein komma verwendet was beim übernehmen in der Konfiguration zum Fehler führt. Es muss ein Punkt sein.
Added 0 tag to current release builds
Fixes #2216
Makefile Show resolved Hide resolved
templates/definition/vehicle/dacia.yaml Show resolved Hide resolved
@andig
Copy link
Member

andig commented Jan 6, 2022

/rebase

@goebelmeier goebelmeier requested a review from andig January 6, 2022 09:33
@andig
Copy link
Member

andig commented Jan 6, 2022

LGTM @DerAndereAndi ?

@DerAndereAndi
Copy link
Contributor

Der change im dacia template ist noch drin.

Ist die einzige Änderung jetzt Umbenamung von "latest" zu "nightly" ? Ich dachte der Hauptpunkt war dass man ohne das moving Tag 0 nicht einfach auf ein neues Release wechseln kann?

@andig
Copy link
Member

andig commented Jan 6, 2022

Ich dachte der Hauptpunkt war dass man ohne das moving Tag 0 nicht einfach auf ein neues Release wechseln kann?

Das Tag ist immer latest, geht also. Dacia löst sich beim Merge auf.

@andig andig merged commit 5fa0a3a into evcc-io:master Jan 6, 2022
@andig
Copy link
Member

andig commented Jan 6, 2022

@DerAndereAndi das könnten wir bloggen- ab jetzt ist latest konservativer und bleeding edge gibts nur noch als nightly.

@DerAndereAndi
Copy link
Contributor

Würde sagen mit dem nächsten Release zusammen

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
infrastructure Basic functionality
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Proper Docker Tagging
4 participants