paulononato/backstage
1
0
Fork 0
Code Issues Pull Requests Actions 36 Packages Projects Releases Wiki Activity

Add squash merge commit title and message options

Browse Source 
Signed-off-by: Kyle Leonhard <kyle.leonhard@snowflake.com>
This commit is contained in:
Kyle Leonhard
2022-12-13 13:48:48 -08:00
parent 28e40488ec
commit a69664faee
8 changed files with 68 additions and 0 deletions
Download Patch File Download Diff File Expand all files Collapse all files
.changeset/spicy-lions-dream.md
+5
View File
@@ -0,0 +1,5 @@
---
'@backstage/plugin-scaffolder-backend': minor
---
Add Github repository support for squash merge commit title and message options
plugins/scaffolder-backend/api-report.md
+4
View File
@@ -195,6 +195,8 @@ export function createGithubRepoCreateAction(options: {
gitAuthorEmail?: string | undefined;
allowRebaseMerge?: boolean | undefined;
allowSquashMerge?: boolean | undefined;
squashMergeCommitTitle?: 'PR_TITLE' | 'COMMIT_OR_PR_TITLE' | undefined,
squashMergeCommitMessage?: 'PR_BODY' | 'COMMIT_MESSAGES' | 'BLANK' | undefined,
allowMergeCommit?: boolean | undefined;
allowAutoMerge?: boolean | undefined;
requireCodeOwnerReviews?: boolean | undefined;
@@ -378,6 +380,8 @@ export function createPublishGithubAction(options: {
gitAuthorEmail?: string | undefined;
allowRebaseMerge?: boolean | undefined;
allowSquashMerge?: boolean | undefined;
squashMergeCommitTitle?: 'PR_TITLE' | 'COMMIT_OR_PR_TITLE' | undefined,
squashMergeCommitMessage?: 'PR_BODY' | 'COMMIT_MESSAGES' | 'BLANK' | undefined,
allowMergeCommit?: boolean | undefined;
allowAutoMerge?: boolean | undefined;
sourcePath?: string | undefined;
plugins/scaffolder-backend/src/scaffolder/actions/builtin/github/githubRepoCreate.test.ts
+12
View File
@@ -106,6 +106,8 @@ describe('github:repo:create', () => {
private: true,
delete_branch_on_merge: false,
allow_squash_merge: true,
squash_merge_commit_title: 'PR_TITLE',
squash_merge_commit_message: 'COMMIT_MESSAGES',
allow_merge_commit: true,
allow_rebase_merge: true,
allow_auto_merge: false,
@@ -126,6 +128,8 @@ describe('github:repo:create', () => {
private: false,
delete_branch_on_merge: false,
allow_squash_merge: true,
squash_merge_commit_title: 'PR_TITLE',
squash_merge_commit_message: 'COMMIT_MESSAGES',
allow_merge_commit: true,
allow_rebase_merge: true,
allow_auto_merge: false,
@@ -147,6 +151,8 @@ describe('github:repo:create', () => {
private: true,
delete_branch_on_merge: false,
allow_squash_merge: true,
squash_merge_commit_title: 'PR_TITLE',
squash_merge_commit_message: 'COMMIT_MESSAGES',
allow_merge_commit: true,
allow_rebase_merge: true,
allow_auto_merge: false,
@@ -172,6 +178,8 @@ describe('github:repo:create', () => {
private: true,
delete_branch_on_merge: false,
allow_squash_merge: true,
squash_merge_commit_title: 'PR_TITLE',
squash_merge_commit_message: 'COMMIT_MESSAGES',
allow_merge_commit: true,
allow_rebase_merge: true,
allow_auto_merge: false,
@@ -192,6 +200,8 @@ describe('github:repo:create', () => {
private: false,
delete_branch_on_merge: false,
allow_squash_merge: true,
squash_merge_commit_title: 'PR_TITLE',
squash_merge_commit_message: 'COMMIT_MESSAGES',
allow_merge_commit: true,
allow_rebase_merge: true,
allow_auto_merge: false,
@@ -213,6 +223,8 @@ describe('github:repo:create', () => {
private: true,
delete_branch_on_merge: false,
allow_squash_merge: true,
squash_merge_commit_title: 'PR_TITLE',
squash_merge_commit_message: 'COMMIT_MESSAGES',
allow_merge_commit: true,
allow_rebase_merge: true,
allow_auto_merge: false,
plugins/scaffolder-backend/src/scaffolder/actions/builtin/github/githubRepoCreate.ts
+8
View File
@@ -49,6 +49,8 @@ export function createGithubRepoCreateAction(options: {
gitAuthorEmail?: string;
allowRebaseMerge?: boolean;
allowSquashMerge?: boolean;
squashMergeCommitTitle?: 'PR_TITLE' | 'COMMIT_OR_PR_TITLE';
squashMergeCommitMessage?: 'PR_BODY' | 'COMMIT_MESSAGES' | 'BLANK';
allowMergeCommit?: boolean;
allowAutoMerge?: boolean;
requireCodeOwnerReviews?: boolean;
@@ -97,6 +99,8 @@ export function createGithubRepoCreateAction(options: {
deleteBranchOnMerge: inputProps.deleteBranchOnMerge,
allowMergeCommit: inputProps.allowMergeCommit,
allowSquashMerge: inputProps.allowSquashMerge,
squashMergeCommitTitle: inputProps.squashMergeCommitTitle,
squashMergeCommitMessage: inputProps.squashMergeCommitMessage,
allowRebaseMerge: inputProps.allowRebaseMerge,
allowAutoMerge: inputProps.allowAutoMerge,
collaborators: inputProps.collaborators,
@@ -122,6 +126,8 @@ export function createGithubRepoCreateAction(options: {
deleteBranchOnMerge = false,
allowMergeCommit = true,
allowSquashMerge = true,
squashMergeCommitTitle = 'PR_TITLE',
squashMergeCommitMessage = 'COMMIT_MESSAGES',
allowRebaseMerge = true,
allowAutoMerge = false,
collaborators,
@@ -153,6 +159,8 @@ export function createGithubRepoCreateAction(options: {
deleteBranchOnMerge,
allowMergeCommit,
allowSquashMerge,
squashMergeCommitTitle,
squashMergeCommitMessage,
allowRebaseMerge,
allowAutoMerge,
access,
plugins/scaffolder-backend/src/scaffolder/actions/builtin/github/helpers.ts
+6
View File
@@ -102,6 +102,8 @@ export async function createGithubRepoWithCollaboratorsAndTopics(
deleteBranchOnMerge: boolean,
allowMergeCommit: boolean,
allowSquashMerge: boolean,
squashMergeCommitTitle: 'PR_TITLE' | 'COMMIT_OR_PR_TITLE' | undefined,
squashMergeCommitMessage: 'PR_BODY' | 'COMMIT_MESSAGES' | 'BLANK' | undefined,
allowRebaseMerge: boolean,
allowAutoMerge: boolean,
access: string | undefined,
@@ -141,6 +143,8 @@ export async function createGithubRepoWithCollaboratorsAndTopics(
delete_branch_on_merge: deleteBranchOnMerge,
allow_merge_commit: allowMergeCommit,
allow_squash_merge: allowSquashMerge,
squash_merge_commit_title: squashMergeCommitTitle,
squash_merge_commit_message: squashMergeCommitMessage,
allow_rebase_merge: allowRebaseMerge,
allow_auto_merge: allowAutoMerge,
homepage: homepage,
@@ -152,6 +156,8 @@ export async function createGithubRepoWithCollaboratorsAndTopics(
delete_branch_on_merge: deleteBranchOnMerge,
allow_merge_commit: allowMergeCommit,
allow_squash_merge: allowSquashMerge,
squash_merge_commit_title: squashMergeCommitTitle,
squash_merge_commit_message: squashMergeCommitMessage,
allow_rebase_merge: allowRebaseMerge,
allow_auto_merge: allowAutoMerge,
homepage: homepage,
plugins/scaffolder-backend/src/scaffolder/actions/builtin/github/inputProperties.ts
+13
View File
@@ -88,6 +88,17 @@ const allowSquashMerge = {
type: 'boolean',
description: `Allow squash merges. The default value is 'true'`,
};
const squashMergeCommitTitle = {
title: 'Default squash merge commit title',
enum: ['PR_TITLE', 'COMMIT_OR_PR_TITLE'],
description: `Sets the default value for a squash merge commit title. The default value is 'PR_TITLE'`,
};
const squashMergeCommitMessage = {
title: 'Default squash merge commit message',
enum: ['PR_BODY', 'COMMIT_MESSAGES', 'BLANK'],
description: `Sets the default value for a squash merge commit message. The default value is 'COMMIT_MESSAGES'`,
};
const allowRebaseMerge = {
title: 'Allow Rebase Merges',
type: 'boolean',
@@ -198,6 +209,8 @@ export { access };
export { allowMergeCommit };
export { allowRebaseMerge };
export { allowSquashMerge };
export { squashMergeCommitTitle };
export { squashMergeCommitMessage };
export { allowAutoMerge };
export { collaborators };
export { defaultBranch };
plugins/scaffolder-backend/src/scaffolder/actions/builtin/publish/github.test.ts
+12
View File
@@ -111,6 +111,8 @@ describe('publish:github', () => {
private: true,
delete_branch_on_merge: false,
allow_squash_merge: true,
squash_merge_commit_title: 'PR_TITLE',
squash_merge_commit_message: 'COMMIT_MESSAGES',
allow_merge_commit: true,
allow_rebase_merge: true,
allow_auto_merge: false,
@@ -131,6 +133,8 @@ describe('publish:github', () => {
private: false,
delete_branch_on_merge: false,
allow_squash_merge: true,
squash_merge_commit_title: 'PR_TITLE',
squash_merge_commit_message: 'COMMIT_MESSAGES',
allow_merge_commit: true,
allow_rebase_merge: true,
allow_auto_merge: false,
@@ -152,6 +156,8 @@ describe('publish:github', () => {
private: true,
delete_branch_on_merge: false,
allow_squash_merge: true,
squash_merge_commit_title: 'PR_TITLE',
squash_merge_commit_message: 'COMMIT_MESSAGES',
allow_merge_commit: true,
allow_rebase_merge: true,
allow_auto_merge: false,
@@ -177,6 +183,8 @@ describe('publish:github', () => {
private: true,
delete_branch_on_merge: false,
allow_squash_merge: true,
squash_merge_commit_title: 'PR_TITLE',
squash_merge_commit_message: 'COMMIT_MESSAGES',
allow_merge_commit: true,
allow_rebase_merge: true,
allow_auto_merge: false,
@@ -197,6 +205,8 @@ describe('publish:github', () => {
private: false,
delete_branch_on_merge: false,
allow_squash_merge: true,
squash_merge_commit_title: 'PR_TITLE',
squash_merge_commit_message: 'COMMIT_MESSAGES',
allow_merge_commit: true,
allow_rebase_merge: true,
allow_auto_merge: false,
@@ -218,6 +228,8 @@ describe('publish:github', () => {
private: true,
delete_branch_on_merge: false,
allow_squash_merge: true,
squash_merge_commit_title: 'PR_TITLE',
squash_merge_commit_message: 'COMMIT_MESSAGES',
allow_merge_commit: true,
allow_rebase_merge: true,
allow_auto_merge: false,
plugins/scaffolder-backend/src/scaffolder/actions/builtin/publish/github.ts
+8
View File
@@ -57,6 +57,8 @@ export function createPublishGithubAction(options: {
gitAuthorEmail?: string;
allowRebaseMerge?: boolean;
allowSquashMerge?: boolean;
squashMergeCommitTitle?: 'PR_TITLE' | 'COMMIT_OR_PR_TITLE';
squashMergeCommitMessage?: 'PR_BODY' | 'COMMIT_MESSAGES' | 'BLANK';
allowMergeCommit?: boolean;
allowAutoMerge?: boolean;
sourcePath?: string;
@@ -117,6 +119,8 @@ export function createPublishGithubAction(options: {
gitAuthorEmail: inputProps.gitAuthorEmail,
allowMergeCommit: inputProps.allowMergeCommit,
allowSquashMerge: inputProps.allowSquashMerge,
squashMergeCommitTitle: inputProps.squashMergeCommitTitle,
squashMergeCommitMessage: inputProps.squashMergeCommitMessage,
allowRebaseMerge: inputProps.allowRebaseMerge,
allowAutoMerge: inputProps.allowAutoMerge,
sourcePath: inputProps.sourcePath,
@@ -154,6 +158,8 @@ export function createPublishGithubAction(options: {
gitAuthorEmail,
allowMergeCommit = true,
allowSquashMerge = true,
squashMergeCommitTitle = 'PR_TITLE',
squashMergeCommitMessage = 'COMMIT_MESSAGES',
allowRebaseMerge = true,
allowAutoMerge = false,
collaborators,
@@ -185,6 +191,8 @@ export function createPublishGithubAction(options: {
deleteBranchOnMerge,
allowMergeCommit,
allowSquashMerge,
squashMergeCommitTitle,
squashMergeCommitMessage,
allowRebaseMerge,
allowAutoMerge,
access,