Merge branch 'main' into dependabot/github_actions/pre-commit-ci/lite-action-1.0.3

Author: rodneylab

.github/workflows/dependency-review.yml

@@ -15,7 +15,7 @@ jobs:
     runs-on: ubuntu-latest
     steps:
       - name: Harden Runner
-        uses: step-security/harden-runner@5c7944e73c4c2a096b17a9cb74d65b6c2bbafbde # v2.9.1
+        uses: step-security/harden-runner@91182cccc01eb5e619899d80e4e971d6181294a7 # v2.10.1
         with:
           egress-policy: audit
       - name: 'Checkout Repository'

.github/workflows/general.yml

@@ -18,7 +18,7 @@ jobs:
     runs-on: ubuntu-latest
     steps:
       - name: Harden Runner
-        uses: step-security/harden-runner@5c7944e73c4c2a096b17a9cb74d65b6c2bbafbde # v2.9.1
+        uses: step-security/harden-runner@91182cccc01eb5e619899d80e4e971d6181294a7 # v2.10.1
         with:
           egress-policy: audit
       - uses: actions/checkout@692973e3d937129bcbf40652eb9f2f61becf3332 # v4.1.7
@@ -30,7 +30,7 @@ jobs:
     runs-on: ubuntu-latest
     steps:
       - name: Harden Runner
-        uses: step-security/harden-runner@5c7944e73c4c2a096b17a9cb74d65b6c2bbafbde # v2.9.1
+        uses: step-security/harden-runner@91182cccc01eb5e619899d80e4e971d6181294a7 # v2.10.1
         with:
           egress-policy: audit
       - uses: actions/checkout@692973e3d937129bcbf40652eb9f2f61becf3332 # v4.1.7
@@ -43,7 +43,7 @@ jobs:
     runs-on: ubuntu-latest
     steps:
       - name: Harden Runner
-        uses: step-security/harden-runner@5c7944e73c4c2a096b17a9cb74d65b6c2bbafbde # v2.9.1
+        uses: step-security/harden-runner@91182cccc01eb5e619899d80e4e971d6181294a7 # v2.10.1
         with:
           egress-policy: audit
       - uses: actions/checkout@692973e3d937129bcbf40652eb9f2f61becf3332 # v4.1.7
@@ -53,7 +53,7 @@ jobs:
     runs-on: ubuntu-latest
     steps:
       - name: Harden Runner
-        uses: step-security/harden-runner@5c7944e73c4c2a096b17a9cb74d65b6c2bbafbde # v2.9.1
+        uses: step-security/harden-runner@91182cccc01eb5e619899d80e4e971d6181294a7 # v2.10.1
         with:
           egress-policy: audit
       - uses: actions/checkout@692973e3d937129bcbf40652eb9f2f61becf3332 # v4.1.7
@@ -67,7 +67,7 @@ jobs:
     runs-on: ubuntu-latest
     steps:
       - name: Harden Runner
-        uses: step-security/harden-runner@5c7944e73c4c2a096b17a9cb74d65b6c2bbafbde # v2.9.1
+        uses: step-security/harden-runner@91182cccc01eb5e619899d80e4e971d6181294a7 # v2.10.1
         with:
           egress-policy: audit
       - name: Install Linux Dependencies

.github/workflows/pre-commit.yml

@@ -11,7 +11,7 @@ jobs:
     runs-on: ubuntu-latest
     steps:
       - name: Harden Runner
-        uses: step-security/harden-runner@5c7944e73c4c2a096b17a9cb74d65b6c2bbafbde # v2.9.1
+        uses: step-security/harden-runner@91182cccc01eb5e619899d80e4e971d6181294a7 # v2.10.1
         with:
           egress-policy: audit
       - uses: actions/checkout@692973e3d937129bcbf40652eb9f2f61becf3332 # v4.1.7

.github/workflows/scorecard.yml

@@ -29,7 +29,7 @@ jobs:
       # actions: read
     steps:
       - name: Harden Runner
-        uses: step-security/harden-runner@5c7944e73c4c2a096b17a9cb74d65b6c2bbafbde # v2.9.1
+        uses: step-security/harden-runner@91182cccc01eb5e619899d80e4e971d6181294a7 # v2.10.1
         with:
           egress-policy: audit
       - name: "Checkout code"

src/lib.rs

@@ -92,7 +92,8 @@ fn strip_trailing_sentence_stub(text: &str) -> (&str, usize) {
     }
 
     if let Some(value) =
-        text[..].rfind(|val: char| val == '.' || val == '\n' || val == '!' || val == '?')
+        //text[..].rfind(|val: char| val == '.' || val == '\n' || val == '!' || val == '?')
+        text[..].rfind(['.', '\n', '!', '?'])
     {
         // last character as a &str
         let last = &text[value..=value];

src/markdown/mod.rs

@@ -114,32 +114,29 @@ pub fn parse_markdown_to_html(
     let mut parsing_heading = false;
     let mut word_count: u32 = 0;
 
-    let heading_parser = Parser::new_ext(markdown, options).map(|event| {
-        match &event {
-            Event::Start(Tag::Heading { .. }) => {
-                parsing_heading = true;
-            }
-            Event::Text(value) => {
-                word_count += words(value);
-                if parsing_heading {
-                    current_id_fragments.push_str(value);
-                }
-            }
-            Event::Code(value) => {
-                if parsing_heading {
-                    current_id_fragments.push_str(value);
-                }
+    let heading_parser = Parser::new_ext(markdown, options).inspect(|event| match event {
+        Event::Start(Tag::Heading { .. }) => {
+            parsing_heading = true;
+        }
+        Event::Text(value) => {
+            word_count += words(value);
+            if parsing_heading {
+                current_id_fragments.push_str(value);
             }
-            Event::End(TagEnd::Heading(_heading_level)) => {
-                let heading = &current_id_fragments;
-                let id = slugified_title(&current_id_fragments);
-                headings.push(Heading::new(heading, &id));
-                current_id_fragments = String::new();
-                parsing_heading = false;
+        }
+        Event::Code(value) => {
+            if parsing_heading {
+                current_id_fragments.push_str(value);
             }
-            _ => {}
         }
-        event
+        Event::End(TagEnd::Heading(_heading_level)) => {
+            let heading = &current_id_fragments;
+            let id = slugified_title(&current_id_fragments);
+            headings.push(Heading::new(heading, &id));
+            current_id_fragments = String::new();
+            parsing_heading = false;
+        }
+        _ => {}
     });
     html::write_html(Cursor::new(&mut bytes), heading_parser)?;
     let reading_time = reading_time_from_words(word_count);