fix first/last commit

Author: lehmanju

.github/fixtures/test-submodules/expected.md

@@ -17,6 +17,12 @@ All notable changes to this project will be documented in this file.
 
 ### Ratatui
 
+#### Bug Fixes
+
+- Run the correct example for chart (#1679)
+- `Rect::positions()` should be empty when width is 0 and height is nonzero (#1669)
+- Ensure that example projects are not published (#1672)
+
 #### Continuous Integration
 
 - Add workflow_dispatch trigger for release-plz (#1693)
@@ -27,11 +33,19 @@ All notable changes to this project will be documented in this file.
 - Bump serde from 1.0.217 to 1.0.218 (#1688)
 - Bump clap from 4.5.29 to 4.5.31 (#1686)
 - Bump serde_json from 1.0.138 to 1.0.139 (#1687)
+- Bump document-features from 0.2.10 to 0.2.11 (#1677)
+- Bump clap from 4.5.28 to 4.5.29 (#1676)
 
 #### Chore
 
 - Release (#1671)
 - Bump termion to 4.0.4 (#1691)
+- Bump termwiz from 0.22.0 to 0.23.0 (#1682)
+
+#### Docs
+
+- Remove link to Paragraph widget (#1683)
+- Update app examples with tapes (#1673)
 
 
 ## [0.1.0] - 2022-04-06

.github/workflows/test-fixtures.yml

@@ -108,6 +108,7 @@ jobs:
             command: v2.6.1..v2.7.0 --include-path .github/fixtures/
           - fixtures-name: test-require-conventional-negative
           - fixtures-name: test-require-conventional-skipped
+          - fixtures-name: test-submodules
 
     steps:
       - name: Checkout

git-cliff/src/lib.rs

@@ -88,15 +88,14 @@ fn process_submodules(
 ) -> Result<()> {
 	// Retrieve first and last commit of a release to create a commit range.
 	let first_commit = release
-		.commits
-		.first()
-		.map(|commit| &commit.id)
-		.and_then(|commit_id| repository.find_commit(commit_id));
+		.previous
+		.as_ref()
+		.and_then(|previous_release| previous_release.commit_id.clone())
+		.and_then(|commit_id| repository.find_commit(&commit_id));
 	let last_commit = release
-		.commits
-		.last()
-		.map(|commit| &commit.id)
-		.and_then(|commit_id| repository.find_commit(commit_id));
+		.commit_id
+		.clone()
+		.and_then(|commit_id| repository.find_commit(&commit_id));
 	let commit_range = first_commit.zip(last_commit);
 
 	let mut submodule_map: HashMap<String, Vec<Commit>> = HashMap::new();
@@ -336,10 +335,6 @@ fn process_repository<'a>(
 		release.commits.push(commit);
 		release.repository = Some(repository_path.clone());
 		if let Some(tag) = tags.get(&commit_id) {
-			if recurse_submodules {
-				process_submodules(repository, release)?;
-			}
-
 			release.version = Some(tag.name.to_string());
 			release.message.clone_from(&tag.message);
 			release.commit_id = Some(commit_id);
@@ -360,6 +355,9 @@ fn process_repository<'a>(
 			previous_release.previous = None;
 			release.previous = Some(Box::new(previous_release));
 			previous_release = release.clone();
+			if recurse_submodules {
+				process_submodules(repository, release)?;
+			}
 			releases.push(Release::default());
 		}
 	}