diff --git a/src/fw/applib/ui/dialogs/expandable_dialog.h b/src/fw/applib/ui/dialogs/expandable_dialog.h
index 594a5687..43434ebf 100644
--- a/src/fw/applib/ui/dialogs/expandable_dialog.h
+++ b/src/fw/applib/ui/dialogs/expandable_dialog.h
@@ -88,7 +88,7 @@ void expandable_dialog_init(ExpandableDialog *expandable_dialog, const char *dia
 //! @return \ref Dialog
 Dialog *expandable_dialog_get_dialog(ExpandableDialog *expandable_dialog);
 
-//! Sets whether or not the expandable dialog should should show its action bar.
+//! Sets whether or not the expandable dialog should show its action bar.
 //! @param expandable_dialog Pointer to the \ref ExpandableDialog to set on
 //! @param show_action_bar Boolean indicating whether to show the action bar
 void expandable_dialog_show_action_bar(ExpandableDialog *expandable_dialog,
diff --git a/tests/fw/test_heap.c b/tests/fw/test_heap.c
index 74bef85e..f62a3d32 100644
--- a/tests/fw/test_heap.c
+++ b/tests/fw/test_heap.c
@@ -234,7 +234,7 @@ void test_heap___heap_bytes_free(void) {
 
   int after_available = heap_bytes_free();
 
-  // make sure the two values are within 16 bytes (usually shoule be 0-8, but 16 for safety)
+  // make sure the two values are within 16 bytes (usually should be 0-8, but 16 for safety)
   cl_assert(abs((before_available - malloc_size_bytes) - after_available) < 16);
 
   heap_free(heap, ptr, 0);
@@ -263,7 +263,7 @@ void test_heap__heap_bytes_used(void) {
 
   int after_used = heap_bytes_used();
 
-  // make sure the two values are within 16 bytes (usually shoule be 0-8, but 16 for safety)
+  // make sure the two values are within 16 bytes (usually should be 0-8, but 16 for safety)
   cl_assert(abs((before_used + malloc_size_bytes) - (after_used)) < 16);
 
   heap_free(heap, ptr, 0);