Skip to content

Commit ad4760b

Browse files
committed
Error cleanup
1 parent bca5643 commit ad4760b

File tree

2 files changed

+10
-8
lines changed

2 files changed

+10
-8
lines changed

buildpacks/nodejs-pnpm-install/src/errors.rs

Lines changed: 9 additions & 7 deletions
Original file line numberDiff line numberDiff line change
@@ -142,23 +142,25 @@ fn on_buildpack_error(error: PnpmInstallBuildpackError) -> ErrorMessage {
142142
PnpmInstallBuildpackError::NodeBuildScriptsMetadata(e) => {
143143
let NodeBuildScriptsMetadataError::InvalidEnabledValue(value) = e;
144144
let value_type = value.type_str();
145+
let requires_metadata = style::value("[requires.metadata]");
146+
let buildplan_name = style::value(NODE_BUILD_SCRIPTS_BUILD_PLAN_NAME);
145147
error_message()
146148
.error_type(ErrorType::UserFacing(
147149
SuggestRetryBuild::No,
148150
SuggestSubmitIssue::Yes,
149151
))
150-
.header("Invalid build script metadata")
151-
.body(formatdoc! { "
152-
A participating buildpack has set invalid `[requires.metadata]` for the build plan \
153-
named `{NODE_BUILD_SCRIPTS_BUILD_PLAN_NAME}`.
154-
152+
.header("Invalid build plan metadata")
153+
.body(formatdoc! {"
154+
A participating buildpack has set invalid {requires_metadata} for the \
155+
build plan named {buildplan_name}.
156+
155157
Expected metadata format:
156158
[requires.metadata]
157159
enabled = <bool>
158-
160+
159161
But was:
160162
[requires.metadata]
161-
enabled = <{value_type}>
163+
enabled = <{value_type}>
162164
"})
163165
.create()
164166
}

buildpacks/nodejs-pnpm-install/src/snapshots/errors___pnpm_install_node_build_scripts_metadata_error.snap

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -1,7 +1,7 @@
11
---
22
source: buildpacks/nodejs-pnpm-install/src/errors.rs
33
---
4-
! Invalid build script metadata
4+
! Invalid build plan metadata
55
!
66
! A participating buildpack has set invalid `[requires.metadata]` for the build plan named `node_build_scripts`.
77
!

0 commit comments

Comments
 (0)