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

kcl on a dir doesn't lazy eval properly #1253

Closed
steeling opened this issue Apr 28, 2024 · 1 comment · Fixed by #1104
Closed

kcl on a dir doesn't lazy eval properly #1253

steeling opened this issue Apr 28, 2024 · 1 comment · Fixed by #1104
Assignees
Labels
enhancement New feature or request

Comments

@steeling
Copy link

Bug Report

1. Minimal reproduce step (Required)

test/versions.k:

version = "v1.0.0"

in test/main.k:

app = {
   version = version   
}

then try again and rename version.k so that it lexicographically comes before main.k, ie: test/_versions.k

2. What did you expect to see? (Required)

version: v1.0.0
app:
  version: v1.0.0

due to proper lazy eval

3. What did you see instead (Required)

app: {}
version: v1.0.0

4. What is your KCL components version? (Required)

kcl version
0.8.3-darwin-arm64

@Peefy
Copy link
Contributor

Peefy commented Apr 28, 2024

Thanks for the feedback. This is a WIP feature. #1104

It is expected to be in v0.9

@Peefy Peefy added the enhancement New feature or request label Apr 29, 2024
@Peefy Peefy self-assigned this Apr 30, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
enhancement New feature or request
Projects
None yet
Development

Successfully merging a pull request may close this issue.

2 participants