vercel · anthonyshew · Jan 18, 2025 · Jan 11, 2025 · Jan 11, 2025 · Jan 11, 2025
@@ -27,9 +27,9 @@ static REQUEST_TIMEOUT: Duration = Duration::from_secs(10);
 
 #[derive(Debug, Error)]
 pub enum Error {
-    #[error("Failed to send analytics event")]
+    #[error("Failed to send analytics event.")]
     SendError(#[from] mpsc::error::SendError<AnalyticsEvent>),
-    #[error("Failed to record analytics")]
+    #[error("Failed to record analytics.")]
     Join(#[from] JoinError),
 }
 

@@ -11,17 +11,17 @@ pub enum Error {
     ReadError(#[from] std::io::Error),
     #[error("Error making HTTP request: {0}")]
     ReqwestError(#[from] reqwest::Error),
-    #[error("skipping HTTP Request, too many failures have occurred.\nLast error: {0}")]
+    #[error("Skipping HTTP Request. Too many failures have occurred.\nLast error: {0}")]
     TooManyFailures(#[from] Box<reqwest::Error>),
     #[error("Unable to set up TLS.")]
     TlsError(#[source] reqwest::Error),
     #[error("Error parsing header: {0}")]
     InvalidHeader(#[from] ToStrError),
     #[error("Error parsing '{url}' as URL: {err}")]
     InvalidUrl { url: String, err: url::ParseError },
-    #[error("unknown caching status: {0}")]
+    #[error("Unknown caching status: {0}")]
     UnknownCachingStatus(String, #[backtrace] Backtrace),
-    #[error("unknown status {code}: {message}")]
+    #[error("Unknown status {code}: {message}")]
     UnknownStatus {
         code: String,
         message: String,

@@ -901,7 +901,7 @@ mod test {
                 .unwrap(),
         );
         let err = APIClient::handle_403(response).await;
-        assert_snapshot!(err.to_string(), @"unknown status forbidden: Not authorized");
+        assert_snapshot!(err.to_string(), @"Unknown status forbidden: Not authorized");
     }
 
     #[tokio::test]

@@ -13,7 +13,7 @@ pub enum Error {
     APIError(#[from] turborepo_api_client::Error),
 
     #[error(
-        "loginUrl is configured to \"{value}\", but cannot be a base URL. This happens in \
+        "`loginUrl` is configured to \"{value}\", but cannot be a base URL. This happens in \
          situations like using a `data:` URL."
     )]
     LoginUrlCannotBeABase { value: String },

@@ -7,39 +7,41 @@ use crate::{CacheActions, CacheConfig};
 
 #[derive(Debug, Error, Diagnostic, PartialEq)]
 pub enum Error {
-    #[error("keys cannot be duplicated, found `{key}` multiple times")]
+    #[error("Keys cannot be duplicated. Found `{key}` multiple times.")]
     DuplicateKeys {
         #[source_code]
         text: String,
         key: &'static str,
         #[label]
         span: Option<SourceSpan>,
     },
-    #[error("actions cannot be duplicated, found `{action}` multiple times")]
+    #[error("Actions cannot be duplicated. Found `{action}` multiple times.")]
     DuplicateActions {
         #[source_code]
         text: String,
         action: &'static str,
         #[label]
         span: Option<SourceSpan>,
     },
-    #[error("invalid cache type and action pair, found `{pair}`, expected colon separated pair")]
+    #[error(
+        "Invalid cache type and action pair. Found `{pair}`, but expected colon separated pair."
+    )]
     InvalidCacheTypeAndAction {
         #[source_code]
         text: String,
         pair: String,
         #[label]
         span: Option<SourceSpan>,
     },
-    #[error("invalid cache action `{c}`")]
+    #[error("Invalid cache action: `{c}`")]
     InvalidCacheAction {
         #[source_code]
         text: String,
         c: char,
         #[label]
         span: Option<SourceSpan>,
     },
-    #[error("invalid cache type `{s}`, expected `local` or `remote`")]
+    #[error("Invalid cache type `{s}`. Expected `local` or `remote`")]
     InvalidCacheType {
         #[source_code]
         text: String,

@@ -415,7 +415,7 @@ mod test {
                 backtrace: Backtrace::capture(),
             },
         );
-        assert_snapshot!(err.to_string(), @"failed to contact remote cache: unknown status unknown: Special message");
+        assert_snapshot!(err.to_string(), @"failed to contact remote cache: Unknown status unknown: Special message");
     }
 
     #[test]

@@ -34,6 +34,6 @@ pub fn vercel_config_dir() -> Result<Option<AbsoluteSystemPathBuf>, PathError> {
 
 #[derive(Debug, Error)]
 pub enum Error {
-    #[error("config directory not found")]
+    #[error("Config directory not found.")]
     ConfigDirNotFound,
 }
@@ -154,13 +154,13 @@ impl GlobSet {
 pub enum Error {
     #[error(transparent)]
     CookieError(#[from] CookieError),
-    #[error("failed to send query to globwatcher: {0}")]
+    #[error("Failed to send query to glob watcher: {0}")]
     SendError(#[from] mpsc::error::SendError<CookiedRequest<Query>>),
-    #[error("globwatcher has closed")]
+    #[error("Glob watcher has closed.")]
     Closed,
-    #[error("globwatcher request timed out")]
+    #[error("Glob watcher request timed out.")]
     Timeout(#[from] tokio::time::error::Elapsed),
-    #[error("glob watching is unavailable")]
+    #[error("Glob watching is unavailable.")]
     Unavailable,
 }
 

@@ -91,13 +91,13 @@ impl HashSpec {
 
 #[derive(Error, Debug)]
 pub enum Error {
-    #[error("package hashing encountered an error: {0}")]
+    #[error("Package hashing encountered an error: {0}")]
     HashingError(String),
-    #[error("file hashing is not available: {0}")]
+    #[error("File hashing is not available: {0}")]
     Unavailable(String),
-    #[error("package not found: {} {:?}", .0.package_path, .0.inputs)]
+    #[error("Package not found: {} {:?}", .0.package_path, .0.inputs)]
     UnknownPackage(HashSpec),
-    #[error("unsupported: glob traverses out of the package")]
+    #[error("Unsupported: glob traverses out of the package")]
     UnsupportedGlob,
 }
 

@@ -15,7 +15,7 @@ pub enum Error {
     Path(#[from] turbopath::PathError),
     #[error(transparent)]
     Io(#[from] io::Error),
-    #[error("error walking directory during recursive copy: {0}")]
+    #[error("Error walking directory during recursive copy: {0}")]
     Walk(#[from] ignore::Error),
 }
 

@@ -11,7 +11,7 @@ use thiserror::Error;
 
 #[derive(Debug, Error)]
 pub enum Error {
-    #[error("cyclic dependency detected:\n{0}")]
+    #[error("Cyclic dependency detected:\n{0}")]
     CyclicDependencies(String),
     #[error("{0} depends on itself")]
     SelfDependency(String),
@@ -103,7 +103,7 @@ mod test {
         assert!(result.is_err());
         let err = result.unwrap_err();
         assert_snapshot!(err.to_string(), @r###"
-        cyclic dependency detected:
+        Cyclic dependency detected:
         	d, c, b, a
         "###);
     }

@@ -19,7 +19,7 @@ use crate::{
 
 #[derive(Debug, Error, Diagnostic)]
 pub enum Error {
-    #[error("No command specified")]
+    #[error("No command specified.")]
     NoCommand(#[backtrace] backtrace::Backtrace),
     #[error("{0}")]
     Bin(#[from] bin::Error, #[backtrace] backtrace::Backtrace),

@@ -4,7 +4,7 @@ use thiserror::Error;
 
 #[derive(Debug, Error)]
 pub enum Error {
-    #[error("could not get path to turbo binary: {0}")]
+    #[error("Could not get path to `turbo` binary: {0}")]
     NoCurrentExe(#[from] io::Error),
 }
 

@@ -9,7 +9,7 @@ use crate::{DaemonConnector, DaemonConnectorError};
 
 #[derive(Debug, Error)]
 pub enum Error {
-    #[error("could not get path to turbo binary: {0}")]
+    #[error("Could not get path to `turbo` binary: {0}")]
     NoCurrentExe(#[from] io::Error),
 }
 

@@ -39,35 +39,35 @@ pub enum Error {
     UsageLimit,
     #[error("spending paused")]
     SpendingPaused,
-    #[error("could not find home directory.")]
+    #[error("Could not find home directory.")]
     HomeDirectoryNotFound,
     #[error("User not found. Please login to Turborepo first by running {command}.")]
     TokenNotFound { command: StyledObject<&'static str> },
     // User decided to not link the remote cache
-    #[error("link cancelled")]
+    #[error("Link cancelled.")]
     NotLinking,
-    #[error("canceled")]
+    #[error("Canceled.")]
     UserCanceled(#[source] dialoguer::Error),
-    #[error("could not get user information {0}")]
+    #[error("Could not get user information: {0}")]
     UserNotFound(#[source] turborepo_api_client::Error),
     // We failed to fetch the team for whatever reason
-    #[error("could not get information for team {1}")]
+    #[error("Could not get information for team: {1}")]
     TeamRequest(#[source] turborepo_api_client::Error, String),
     // We fetched the team, but it doesn't exist.
-    #[error("could not find team {0}")]
+    #[error("Could not find team: {0}")]
     TeamNotFound(String),
-    #[error("could not get teams information")]
+    #[error("Could not get teams information.")]
     TeamsRequest(#[source] turborepo_api_client::Error),
-    #[error("could not get spaces information")]
+    #[error("Could not get spaces information.")]
     SpacesRequest(#[source] turborepo_api_client::Error),
-    #[error("could not get caching status")]
+    #[error("Could not get caching status.")]
     CachingStatusNotFound(#[source] turborepo_api_client::Error),
     #[error("Failed to open browser. Please visit {0} to enable Remote Caching")]
     OpenBrowser(String, #[source] io::Error),
-    #[error("please re-run `link` after enabling caching")]
+    #[error("Please re-run `link` after enabling caching.")]
     EnableCaching,
     #[error(
-        "Could not persist selected space ({space_id}) to `experimentalSpaces.id` in turbo.json"
+        "Could not persist selected space ({space_id}) to `experimentalSpaces.id` in turbo.json."
     )]
     WriteToTurboJson {
         space_id: String,

@@ -18,7 +18,7 @@ use crate::{
 
 #[derive(Debug, Error, Diagnostic)]
 pub enum Error {
-    #[error("package `{package}` not found")]
+    #[error("Package `{package}` not found.")]
     PackageNotFound { package: String },
 }
 

@@ -22,13 +22,13 @@ pub const DEFAULT_OUTPUT_DIR: &str = "out";
 
 #[derive(Debug, thiserror::Error, Diagnostic)]
 pub enum Error {
-    #[error("io error while pruning: {0}")]
+    #[error("I/O error while pruning: {0}")]
     Io(#[from] std::io::Error),
     #[error("File system error while pruning. The error from the operating system is: {0}")]
     Fs(#[from] turborepo_fs::Error),
-    #[error("json error while pruning: {0}")]
+    #[error("JSON error while pruning: {0}")]
     Json(#[from] serde_json::Error),
-    #[error("path error while pruning: {0}")]
+    #[error("Path error while pruning: {0}")]
     Path(#[from] turbopath::PathError),
     #[error(transparent)]
     #[diagnostic(transparent)]
@@ -39,15 +39,15 @@ pub enum Error {
     PackageGraph(#[from] package_graph::Error),
     #[error(transparent)]
     Lockfile(#[from] turborepo_lockfiles::Error),
-    #[error("turbo doesn't support workspaces at file system root")]
+    #[error("`turbo` does not support workspaces at file system root.")]
     WorkspaceAtFilesystemRoot,
-    #[error("at least one target must be specified")]
+    #[error("At least one target must be specified.")]
     NoWorkspaceSpecified,
-    #[error("invalid scope: package with name {0} in package.json not found")]
+    #[error("Invalid scope. Package with name {0} in `package.json` not found.")]
     MissingWorkspace(PackageName),
-    #[error("Cannot prune without parsed lockfile")]
+    #[error("Cannot prune without parsed lockfile.")]
     MissingLockfile,
-    #[error("Prune is not supported for Bun")]
+    #[error("`prune` is not supported for Bun.")]
     BunUnsupported,
     #[error("Unable to read config: {0}")]
     Config(#[from] crate::config::Error),
-Original file line number
+Diff line change
@@ Expand Up / @@ -415,7 +415,7 @@ mod test { @@
                     backtrace: Backtrace::capture(),
                 },
             );
-            assert_snapshot!(err.to_string(), @"failed to contact remote cache: unknown status unknown: Special message");
+            assert_snapshot!(err.to_string(), @"failed to contact remote cache: Unknown status unknown: Special message");
         }
         #[test]
@@ Expand Down @@