Skip to content

Commit

Permalink
Removes usage of --full-history
Browse files Browse the repository at this point in the history
 - Unwanted perf overhead for basically no value
  • Loading branch information
eamodio committed Feb 27, 2025
1 parent 70a1f0b commit f0dfaa2
Show file tree
Hide file tree
Showing 2 changed files with 3 additions and 3 deletions.
4 changes: 2 additions & 2 deletions src/env/node/git/sub-providers/commits.ts
Original file line number Diff line number Diff line change
Expand Up @@ -344,7 +344,7 @@ export class CommitsGitSubProvider implements GitCommitsSubProvider {
];

if (options?.status !== false) {
args.push('--name-status', '--full-history');
args.push('--name-status');
}
if (options?.all) {
args.push('--all');
Expand Down Expand Up @@ -487,7 +487,7 @@ export class CommitsGitSubProvider implements GitCommitsSubProvider {

try {
const parser = createLogParserSingle('%H');
const args = [...parser.arguments, '--full-history'];
const args = [...parser.arguments];

const ordering = options?.ordering ?? configuration.get('advanced.commitOrdering');
if (ordering) {
Expand Down
2 changes: 1 addition & 1 deletion src/env/node/git/sub-providers/contributors.ts
Original file line number Diff line number Diff line change
Expand Up @@ -52,7 +52,7 @@ export class ContributorsGitSubProvider implements GitContributorsSubProvider {
const currentUser = await this.provider.config.getCurrentUser(repoPath);
const parser = getContributorsParser(options?.stats);

const args = [...parser.arguments, '--full-history', '--use-mailmap'];
const args = [...parser.arguments, '--use-mailmap'];

const merges = options?.merges ?? true;
if (merges) {
Expand Down

0 comments on commit f0dfaa2

Please sign in to comment.