anonaddy / docker

AnonAddy Docker image
MIT License
519 stars 56 forks source link

examples: bump images version #263

Open luckydonald opened 9 months ago

luckydonald commented 9 months ago

Updated dependency containers (traefik, mariadb, redis):

Tested the traefik configuration with anonaddy:1.0.8.

That's also the reason why I didn't bump nginx, as I didn't test that example.
I only updated the tested images.

luckydonald commented 9 months ago

Okey, that failed test build is weird. It says

Error: buildx bake failed with: 
 ERROR: failed to solve: failed to do request: 
 Head "https://registry-1.docker.io/v2/docker/dockerfile/manifests/1": 
 net/http: TLS handshake timeout

Sounds like a hiccup with some docker api?

luckydonald commented 9 months ago

I was under the impression github would have pr-squasing somewhere in the UI. Oh well.

Anyways, it's squashed now.

Even if commit cecce66 was much cooler sounding than 5874c3c.

Edit: Apparently it does: https://docs.github.com/en/repositories/configuring-branches-and-merges-in-your-repository/configuring-pull-request-merges/configuring-commit-squashing-for-pull-requests

Under "Pull Requests", select Allow squash merging. This allows contributors to merge a pull request by squashing all commits into a single commit. The default commit message presented to contributors when merging is the commit title and message if the pull request contains only 1 commit, or the pull request title and list of commits if the pull request contains 2 or more commits.

luckydonald commented 8 months ago

@crazy-max anything missing here for a merge?