Skip to content

Implement jq templating #93

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

Merged
merged 1 commit into from
Oct 20, 2023
Merged

Conversation

LaurentGoderre
Copy link
Member

@LaurentGoderre LaurentGoderre commented Oct 12, 2023

No description provided.

@LaurentGoderre LaurentGoderre force-pushed the templating branch 3 times, most recently from a7ccf38 to 9ab7a0e Compare October 12, 2023 16:35
@LaurentGoderre LaurentGoderre changed the title Implement jq templating, version and OS bump Implement jq templating Oct 13, 2023
Copy link
Member

@yosifkit yosifkit left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Ok, a few review comments.

@@ -1,13 +1,14 @@
FROM alpine:3.18
FROM alpine:{{ env.variant | ltrimstr("alpine") }}
Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

I think a future goal would be to combine the Alpine and Debian templates into one to ensure that the common bits and flow remain in sync. This can be a future PR or added to this one 🤷.

Copy link
Member

@yosifkit yosifkit left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Some indentation irregularities and maybe swap to an array instead of a string.

@yosifkit yosifkit merged commit 1cd4fbf into docker-library:master Oct 20, 2023
yosifkit added a commit to docker-library/oi-janky-groovy that referenced this pull request Oct 20, 2023
docker-library-bot added a commit to docker-library-bot/official-images that referenced this pull request Oct 20, 2023
Changes:

- docker-library/memcached@1cd4fbf: Merge pull request docker-library/memcached#93 from LaurentGoderre/templating
- docker-library/memcached@8b76567: Implement jq templating
@LaurentGoderre LaurentGoderre deleted the templating branch October 23, 2023 14:18
docker-library-bot added a commit to docker-library-bot/official-images that referenced this pull request Dec 9, 2023
Changes:

- docker-library/memcached@471a298: Update 1 to alpine 3.19
- docker-library/memcached@9cd87ed: Merge pull request docker-library/memcached#94 from infosiftr/one-template
- docker-library/memcached@52c3fa2: Update templating to pin to only major version and use a single template
- docker-library/memcached@1cd4fbf: Merge pull request docker-library/memcached#93 from LaurentGoderre/templating
- docker-library/memcached@8b76567: Implement jq templating
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