-
Notifications
You must be signed in to change notification settings - Fork 51
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
kpm run file order issue #241
Closed
steeling opened this issue
Dec 26, 2023
· 4 comments
· Fixed by kcl-lang/kcl#1104 or kcl-lang/kcl#1213
Closed
kpm run file order issue #241
steeling opened this issue
Dec 26, 2023
· 4 comments
· Fixed by kcl-lang/kcl#1104 or kcl-lang/kcl#1213
Comments
Related Issue: kcl-lang/kcl#247 @steeling Hi there, thank you for your feedback. We are considering supporting this feature. The internal scope of the configuration block references global scope variables, which are currently not visible. |
is this issue still relevant? |
Yes. |
can you please assign me? |
16 tasks
16 tasks
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Bug Report
1. Minimal reproduce step (Required)
Create the following files in a directory called
my_app
my_app/base.k
:my_app/input1.k
:my_app/versions.k
:2. What did you expect to see? (Required)
This should compile via
kpm run my_app
. However it can be fixed if we renamemy_app/versions.k
, so that it lexicographically precedesinput1.k
:This now succeeds.
3. What did you see instead (Required)
4. What is your KCL components version? (Required)
kpm version 0.5.0
The text was updated successfully, but these errors were encountered: