From 182a86647000d40cadb32f5c29ed55b58a75e172 Mon Sep 17 00:00:00 2001 From: Alex Coleman Date: Tue, 9 Jan 2024 14:02:34 -0800 Subject: [PATCH] Fix missing space in diagnostic location print (#4573) Summary: Fix missing space in diagnostic location print A space was missing between the error message and location in Diagnostic.print_without_source, which made it harder to open the file from terminal output with tools like vscode. See https://github.com/facebook/relay/issues/4571 Pull Request resolved: https://github.com/facebook/relay/pull/4573 Test Plan: Confirmed that now the path is outputted in a format that vscode can parse out: ``` [ERROR] Compilation failed. [ERROR] Unable to run relay compiler. Error details: Failed to build: - Validation errors: - The type `Company` has no field `metric`. Did you mean `metrics`? See https://relay.dev/docs/error-reference/unknown-field/: client/components/console/pulse/details_dialog/PulseMetricDetailsDialog.tsx:465:471 ``` Note that the line range is still incorrect (it is using the character span instead of linenumber:character), but this seems much trickier to fix. Reviewed By: monicatang Differential Revision: D52635842 Pulled By: captbaritone fbshipit-source-id: fabc0aadb1b72deac2da377a44dd46afed46be60 --- compiler/crates/common/src/diagnostic.rs | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/compiler/crates/common/src/diagnostic.rs b/compiler/crates/common/src/diagnostic.rs index a341f6983e11f..2286b7c351f68 100644 --- a/compiler/crates/common/src/diagnostic.rs +++ b/compiler/crates/common/src/diagnostic.rs @@ -225,7 +225,7 @@ impl Diagnostic { let mut result = String::new(); writeln!( result, - "{message}:{location:?}", + "{message}: {location:?}", message = &self.0.message, location = self.0.location )