Skip to content

Commit b1d5648

Browse files
committed
Use slice instead of splice and add changelog entry
Use args.slice(1) instead of args.splice(1) to avoid unnecessary array mutation. Add changelog entry for new env vars and logging.
1 parent f5330c1 commit b1d5648

2 files changed

Lines changed: 2 additions & 1 deletion

File tree

news/changelog-1.9.md

Lines changed: 1 addition & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -109,6 +109,7 @@ All changes included in 1.9:
109109

110110
## Projects
111111

112+
- ([#12444](https://github.com/quarto-dev/quarto-cli/pull/12444)): Improve pre/post render script logging with `Running script` prefix and add `QUARTO_PROJECT_SCRIPT_PROGRESS` and `QUARTO_PROJECT_SCRIPT_QUIET` environment variables so scripts can adapt their output.
112113
- ([#13892](https://github.com/quarto-dev/quarto-cli/issues/13892)): Fix `output-dir: ./` deleting entire project directory. `output-dir` must be a subdirectory of the project directory and check is now better to avoid deleting the project itself when it revolves to the same path.
113114

114115
### `website`

src/command/render/project.ts

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -986,7 +986,7 @@ async function runScripts(
986986
env["QUARTO_USE_FILE_FOR_PROJECT_OUTPUT_FILES"] = output;
987987
}
988988

989-
const result = await handler.run(script, args.splice(1), undefined, {
989+
const result = await handler.run(script, args.slice(1), undefined, {
990990
cwd: projDir,
991991
stdout: quiet ? "piped" : "inherit",
992992
env,

0 commit comments

Comments
 (0)