feat(git): Specify additional git authors to ignore

docs/usage/configuration-options.md

@@ -635,6 +635,20 @@ If configured, Renovate bypasses its normal major/minor/patch upgrade logic and
 Beware that Renovate follows tags strictly.
 For example, if you are following a tag like `next` and then that stream is released as `stable` and `next` is no longer being updated then that means your dependencies also won't be getting updated.
 
+## gitIgnoredAuthors
+
+Specify commit authors ignored by Renovate.
+
+If you have other bots which commit on top of Renovate PRs, you probably want Renovate to rebase those PRs instead of treating them as modified.
+
+Example:
+
+```json
+{
+  "gitIgnoredAuthors": ["some-bot@example.org"]
+}
+```
+
 ## gitLabAutomerge
 
 Caution (fixed in GitLab >= 12.7): when this option is enabled it is possible due to a bug in GitLab that MRs with failing pipelines might still get merged.

lib/config/definitions.ts

@@ -589,6 +589,14 @@ const options: RenovateOptions[] = [
     admin: true,
     stage: 'global',
   },
+  {
+    name: 'gitIgnoredAuthors',
+    description:
+      'Additional git authors which are ignored by Renovate. Must conform to RFC5322.',
+    type: 'array',
+    subType: 'string',
+    stage: 'repository',
+  },
   {
     name: 'enabledManagers',
     description:

lib/config/types.ts

@@ -61,6 +61,7 @@ export interface RenovateSharedConfig {
   suppressNotifications?: string[];
   timezone?: string;
   unicodeEmoji?: boolean;
+  gitIgnoredAuthors?: string[];
 }
 
 // Config options used only within the global worker

lib/util/git/index.spec.ts

@@ -145,11 +145,15 @@ describe('platform/git', () => {
     it('should return false when branch is not found', async () => {
       expect(await git.isBranchModified('renovate/not_found')).toBe(false);
     });
-    it('should return true when author matches', async () => {
+    it('should return false when author matches', async () => {
       expect(await git.isBranchModified('renovate/future_branch')).toBe(false);
       expect(await git.isBranchModified('renovate/future_branch')).toBe(false);
     });
-    it('should return false when custom author', async () => {
+    it('should return false when author is ignored', async () => {
+      git.setIgnoredAuthors(['custom@example.com']);
+      expect(await git.isBranchModified('renovate/custom_author')).toBe(false);
+    });
+    it('should return true when custom author is unknown', async () => {
       expect(await git.isBranchModified('renovate/custom_author')).toBe(true);
     });
   });

lib/util/git/index.ts

@@ -1,4 +1,5 @@
 import URL from 'url';
+import is from '@sindresorhus/is';
 import fs from 'fs-extra';
 import GitUrlParse from 'git-url-parse';
 import Git, {
@@ -51,6 +52,7 @@ interface LocalConfig extends StorageConfig {
   branchCommits: Record<string, CommitSha>;
   branchIsModified: Record<string, boolean>;
   branchPrefix: string;
+  ignoredAuthors: string[];
 }
 
 // istanbul ignore next
@@ -165,6 +167,7 @@ async function fetchBranchCommits(): Promise<void> {
 
 export async function initRepo(args: StorageConfig): Promise<void> {
   config = { ...args } as any;
+  config.ignoredAuthors = [];
   config.additionalBranches = [];
   config.branchIsModified = {};
   git = Git(config.localDir);
@@ -215,6 +218,13 @@ export async function setBranchPrefix(branchPrefix: string): Promise<void> {
   }
 }
 
+export function setIgnoredAuthors(ignoredAuthors?: string[]): void {
+  config.ignoredAuthors = [];
+  if (is.array(ignoredAuthors)) {
+    config.ignoredAuthors = ignoredAuthors;
+  }
+}
+
 export async function getSubmodules(): Promise<string[]> {
   try {
     return (
@@ -478,7 +488,8 @@ export async function isBranchModified(branchName: string): Promise<boolean> {
   const { gitAuthorEmail } = config;
   if (
     lastAuthor === process.env.RENOVATE_LEGACY_GIT_AUTHOR_EMAIL || // remove in next major release
-    lastAuthor === gitAuthorEmail
+    lastAuthor === gitAuthorEmail ||
+    config.ignoredAuthors.some((ignoredAuthor) => lastAuthor === ignoredAuthor)
   ) {
     // author matches - branch has not been modified
     config.branchIsModified[branchName] = false;

lib/workers/repository/init/index.ts

@@ -1,7 +1,7 @@
 import { RenovateConfig } from '../../../config';
 import { logger } from '../../../logger';
 import { clone } from '../../../util/clone';
-import { setBranchPrefix } from '../../../util/git';
+import { setBranchPrefix, setIgnoredAuthors } from '../../../util/git';
 import { checkIfConfigured } from '../configured';
 import { initApis } from './apis';
 import { initializeCaches } from './cache';
@@ -21,6 +21,7 @@ export async function initRepo(
   config = await getRepoConfig(config);
   checkIfConfigured(config);
   await setBranchPrefix(config.branchPrefix);
+  setIgnoredAuthors(config.gitIgnoredAuthors);
   config = await detectVulnerabilityAlerts(config);
   // istanbul ignore if
   if (config.printConfig) {