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

Create docker-publish.yml #40

Merged
merged 3 commits into from
Dec 4, 2023
Merged

Create docker-publish.yml #40

merged 3 commits into from
Dec 4, 2023

Conversation

jhns-de
Copy link
Collaborator

@jhns-de jhns-de commented Nov 30, 2023

Standard yml um images zu bauen und in der github registry zu veröffentlichen. Funktioniert bei mir in einen Repos super.

Standard yml um images zu bauen und in der github registry zu veröffentlichen. Funktioniert bei mir in einen Repos super.
@jhns-de jhns-de added the docker Pull requests that update Docker code label Nov 30, 2023
@jo-gross
Copy link
Owner

Wenn du lokal herausfinden möchtest, ob das Format korrekt ist, kannst du das mit yarn format:check testen. mit yarn format:fix kannst du bestehende Probleme lösen.

@jhns-de
Copy link
Collaborator Author

jhns-de commented Nov 30, 2023

Wir müssen noch entweder den Master in Main umbenennen oder in dem Skript den Branch auf Master ändern.

@jo-gross
Copy link
Owner

jo-gross commented Dec 4, 2023

@jhns-de welche Schritte müssen realisiert werden, um das Image auf ein Repository zu publishen. Hast du eine Referenz, was genau dazu eingerichtet werden muss?

@jhns-de
Copy link
Collaborator Author

jhns-de commented Dec 4, 2023

Das published das schon direkt in die Github Registry: ghcr.io/jo-gross/Cocktail-Manager:master

Ansonsten kann man hier auch noch mehr einstellen: https://github.com/docker/build-push-action oder hier https://github.com/docker/login-action#github-container-registry

Und Packages wird automatisch aktiviert, sobald man da hin pusht. Sprich nach dem merge sollte alles automatisch funktionieren. Dann kann man sich gedanken machen, dass das irgendwie Cached, um Buildzeit zu sparen.

@jo-gross jo-gross merged commit 746d731 into master Dec 4, 2023
3 checks passed
@jo-gross jo-gross deleted the docker-publish branch December 4, 2023 14:51
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
docker Pull requests that update Docker code
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants