refactor: pull the offline flag on a global level

This will also skip the update check if the flag is enabled.

See #732

src/cmd/build.rs

@@ -32,9 +32,7 @@ pub struct Build {
     #[arg(short, long, env = "TRUNK_BUILD_DIST")]
     pub dist: Option<PathBuf>,
 
-    /// Run without accessing the network
-    #[arg(long, env = "TRUNK_BUILD_OFFLINE")]
-    #[arg(default_missing_value="true", num_args=0..=1)]
+    #[arg(from_global)]
     pub offline: Option<bool>,
 
     /// Require Cargo.lock and cache are up to date

src/main.rs

@@ -132,6 +132,11 @@ struct Trunk {
     #[arg(long, global(true), env = "TRUNK_SKIP_VERSION_CHECK")]
     pub skip_version_check: bool,
 
+    /// Run without accessing the network
+    #[arg(long, global(true), env = "TRUNK_OFFLINE")]
+    #[arg(default_missing_value = "true", num_args=0..=1)]
+    pub offline: Option<bool>,
+
     /// Color mode
     #[arg(long, env = "TRUNK_COLOR", global(true), value_enum, conflicts_with = "no_color", default_value_t = ColorMode::Auto)]
     pub color: ColorMode,
@@ -167,7 +172,7 @@ enum ColorMode {
 impl Trunk {
     #[tracing::instrument(level = "trace", skip(self))]
     pub async fn run(self) -> Result<()> {
-        version::update_check(self.skip_version_check);
+        version::update_check(self.skip_version_check | self.offline.unwrap_or_default());
 
         match self.action {
             TrunkSubcommands::Build(inner) => inner.run(self.config).await,

src/version/enabled/mod.rs

@@ -4,18 +4,22 @@ use crate::version::{
 };
 use semver::Version;
 use std::time::Duration;
+use tracing::instrument;
 
 mod state;
 
+#[instrument]
 pub fn update_check(skip: bool) {
+    tracing::trace!("Update check");
+
     if skip {
         return;
     }
 
     tracing::debug!("Spawning update check");
 
     // We need to spawn this in a dedicated tokio runtime, as otherwise this would block
-    // the current tokio runtime from existing. There seems to be an issue with where even
+    // the current tokio runtime from exiting. There seems to be an issue with where even
     // with an aborted spawned task, tokio will wait for it to end indefinitely.
     std::thread::spawn(|| {
         perform_update_check();