build: Update Security example to 3.1.0 released images.

edgexfoundry / edgex-examples

Apache License 2.0

51 stars 52 forks source link

Closed lenny-goodell closed 8 months ago

lenny-goodell commented 8 months ago

Removed obsolte compose files and fixed doc link in README.

If your build fails due to your commit message not passing the build checks, please review the guidelines here: https://github.com/edgexfoundry/edgex-examples/blob/main/.github/Contributing.md

Please check if your PR fulfills the following requirements:

[ ] I have added unit tests for the new feature or bug fix (if not, why?)
[ ] I have fully tested (add details below) this the new feature or bug fix (if not, why?)

bnevis-i commented 8 months ago

Removed obsolte compose files and fixed doc link in README.

Would you be open to publishing a large context diff instead? I wanted to show what the specific changes where to enable this functionality.

lenny-goodell commented 8 months ago

Would you be open to publishing a large context diff instead?

Not following what you are looking for?

bnevis-i commented 8 months ago

Would you be open to publishing a large context diff instead?

Not following what you are looking for?

diff -c 10 original-docker-compose.yaml new-docker-compose.yaml > docker-compose.diff

and check in docker-compose.diff

The idea is to show people how to recreate the supplied docker-compose.

lenny-goodell commented 8 months ago

The idea is to show people how to recreate the supplied docker-compose.

As part of the tutorial? Not sure the value and it is out of scope for this PR.