Skip to content

Fix: Use variable names from jsonnet stdlib for is_x builtins#94

Merged
JarvisCraft merged 1 commit intoCertainLach:masterfrom
jackatbancast:fixup-is_x-builtins-variable-names
Nov 30, 2022
Merged

Fix: Use variable names from jsonnet stdlib for is_x builtins#94
JarvisCraft merged 1 commit intoCertainLach:masterfrom
jackatbancast:fixup-is_x-builtins-variable-names

Conversation

@jackatbancast
Copy link
Copy Markdown
Contributor

This changes the variable name in the identity-related is_X builtin functions to match the variable names in the jsonnet standard library.

This mainly serves to integrate better with libraries such as k8s-libsonnet.

This changes the variable name in the identity-related `is_X` builtin
functions to match the variable names in the [jsonnet standard
library](https://github.com/google/jsonnet/blob/master/stdlib/std.jsonnet#L28,L33).

This mainly serves to integrate better with libraries such as
[k8s-libsonnet](https://github.com/jsonnet-libs/k8s-libsonnet/).
@JarvisCraft JarvisCraft self-assigned this Nov 30, 2022
@JarvisCraft JarvisCraft added enhancement New feature or request good first issue Good for newcomers labels Nov 30, 2022
@JarvisCraft JarvisCraft added this to the v0.5.0 milestone Nov 30, 2022
@JarvisCraft JarvisCraft merged commit aab1050 into CertainLach:master Nov 30, 2022
@JarvisCraft
Copy link
Copy Markdown
Collaborator

Thanks for your contribution! The change will land in 0.5.0.

@jackatbancast jackatbancast deleted the fixup-is_x-builtins-variable-names branch October 6, 2023 16:17
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

enhancement New feature or request good first issue Good for newcomers

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants