Amend

casey · Jan 22, 2025 · d715cb5 · d715cb5
1 parent 8316daf
commit d715cb5
Show file tree

Hide file tree

Showing 2 changed files with 8 additions and 9 deletions.
diff --git a/src/compile_error.rs b/src/compile_error.rs
@@ -120,8 +120,8 @@ impl Display for CompileError<'_> {
       }
       DuplicateUnexport { variable } => {
         write!(f, "Variable `{variable}` is unexported multiple times")
-      },
-      ExitMessageAndNoExitMessageAttribute { recipe } =>write!(
+      }
+      ExitMessageAndNoExitMessageAttribute { recipe } => write!(
         f,
         "Recipe `{recipe}` has both `[exit-message]` and `[no-exit-message]` attributes"
       ),

diff --git a/src/parser.rs b/src/parser.rs
@@ -951,20 +951,19 @@ impl<'run, 'src> Parser<'run, 'src> {
       }));
     }
 
-    let working_directory = attributes.contains(AttributeDiscriminant::WorkingDirectory);
-
-    if working_directory && attributes.contains(AttributeDiscriminant::NoCd) {
+    if attributes.contains(AttributeDiscriminant::WorkingDirectory)
+      && attributes.contains(AttributeDiscriminant::NoCd)
+    {
       return Err(
         name.error(CompileErrorKind::NoCdAndWorkingDirectoryAttribute {
           recipe: name.lexeme(),
         }),
       );
     }
 
-    let no_exit_message_attr = attributes.contains(AttributeDiscriminant::ExitMessage);
-    let exit_message_attr = attributes.contains(AttributeDiscriminant::NoExitMessage);
-
-    if no_exit_message_attr && exit_message_attr {
+    if attributes.contains(AttributeDiscriminant::ExitMessage)
+      && attributes.contains(AttributeDiscriminant::NoExitMessage)
+    {
       return Err(
         name.error(CompileErrorKind::ExitMessageAndNoExitMessageAttribute {
           recipe: name.lexeme(),