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

feat: Add Sprig template functions #9005

Merged
merged 4 commits into from
Aug 10, 2023

Conversation

briantopping
Copy link
Contributor

Description

Add Sprig template functions. This allows parameterizing with functions such as date

User facing changes (remove if N/A)

Additional functions are available to templated fields. No other changes are made to existing APIs or UX.

Tests for the functions are comprehensively provided upstream.

@conventional-commit-lint-gcf
Copy link

conventional-commit-lint-gcf bot commented Aug 9, 2023

🤖 I detect that the PR title and the commit message differ and there's only one commit. To use the PR title for the commit history, you can use Github's automerge feature with squashing, or use automerge label. Good luck human!

-- conventional-commit-lint bot
https://conventionalcommits.org/

@briantopping briantopping changed the title Add Sprig template functions feat: Add Sprig template functions Aug 9, 2023
@briantopping
Copy link
Contributor Author

cc: @aaron-prindle @ericzzzzzzz

Signed-off-by: Brian Topping <brian.topping@sap.com>
Signed-off-by: Brian Topping <brian.topping@sap.com>
@briantopping
Copy link
Contributor Author

Hi @ericzzzzzzz, I missed the test failure before, should be fixed. Apologies for the noise.

@ericzzzzzzz ericzzzzzzz added the kokoro:run runs the kokoro jobs on a PR label Aug 10, 2023
@kokoro-team kokoro-team removed the kokoro:run runs the kokoro jobs on a PR label Aug 10, 2023
@ericzzzzzzz
Copy link
Contributor

Hi @ericzzzzzzz, I missed the test failure before, should be fixed. Apologies for the noise.

no worries, I just re-triggered the build.

Signed-off-by: Brian Topping <brian.topping@sap.com>
@briantopping
Copy link
Contributor Author

@ericzzzzzzz I guess I am learning what has to happen the hard way on my first PR. Doesn't seem I can add that label to kick off a build.

@ericzzzzzzz ericzzzzzzz added the kokoro:run runs the kokoro jobs on a PR label Aug 10, 2023
@kokoro-team kokoro-team removed the kokoro:run runs the kokoro jobs on a PR label Aug 10, 2023
@ericzzzzzzz ericzzzzzzz added the kokoro:force-run forces a kokoro re-run on a PR label Aug 10, 2023
@kokoro-team kokoro-team removed the kokoro:force-run forces a kokoro re-run on a PR label Aug 10, 2023
@ericzzzzzzz ericzzzzzzz merged commit b4dd9f9 into GoogleContainerTools:main Aug 10, 2023
13 checks passed
@briantopping briantopping deleted the sprig branch August 10, 2023 13:56
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

3 participants