starred/pkl
1
0
Fork 0
mirror of https://github.com/apple/pkl.git synced 2026-04-28 03:07:14 +02:00
Code Issues 198 Packages Projects Releases 10 Wiki Activity

Fix regression in CLI error handling when project loading fails (#1439)

Browse Source
This commit is contained in:
Jen Basch
2026-02-25 01:28:05 -08:00
committed by GitHub
parent 28b09134d7
commit 2e4d73b957
2 changed files with 3 additions and 2 deletions
Download Patch File Download Diff File Expand all files Collapse all files

2
docs/modules/release-notes/pages/0.31.adoc
View File

@@ -196,7 +196,7 @@ image::syntax-highlight-error.png[syntax highlighted output]
[[cli-dependency-notation]] [[cli-dependency-notation]]
=== CLI Support for Dependency Notation === CLI Support for Dependency Notation
The Pkl CLI now supports specifying modules using xref:language-reference:index.adoc#project-dependencies[dependency notation] (pr:https://github.com/apple/pkl/pull/1434[]). The Pkl CLI now supports specifying modules using xref:language-reference:index.adoc#project-dependencies[dependency notation] (pr:https://github.com/apple/pkl/pull/1434[], pr:https://github.com/apple/pkl/pull/1439[]).
This is especially helpful for <<cli-framework,CLI commands>> defined in Packages: This is especially helpful for <<cli-framework,CLI commands>> defined in Packages:
[source,bash] [source,bash]

3
pkl-commons-cli/src/main/kotlin/org/pkl/commons/cli/CliCommand.kt
View File

@@ -111,9 +111,10 @@ abstract class CliCommand(protected val cliOptions: CliBaseOptions) {
remoteDep.packageUri.toPackageAssetUri(path).uri remoteDep.packageUri.toPackageAssetUri(path).uri
} }
protected val resolvedSourceModules: List<URI> = protected val resolvedSourceModules: List<URI> by lazy {
if (project == null) cliOptions.normalizedSourceModules if (project == null) cliOptions.normalizedSourceModules
else cliOptions.normalizedSourceModules.map(::resolveModuleUri) else cliOptions.normalizedSourceModules.map(::resolveModuleUri)
}
protected fun loadProject(projectFile: Path): Project { protected fun loadProject(projectFile: Path): Project {
val securityManager = val securityManager =