From ffbb37fe01a8cb38f4e5ea2da138a827b37d08a0 Mon Sep 17 00:00:00 2001 From: Michael Waddell Date: Fri, 11 Feb 2022 14:08:20 -0600 Subject: [PATCH] renaming `target_branch` to `target-branch` --- README.md | 4 ++-- action.yml | 4 ++-- dist/index.js | 6 +++--- src/dependabot/output.test.ts | 8 ++++---- src/dependabot/output.ts | 4 ++-- src/main.test.ts | 4 ++-- 6 files changed, 15 insertions(+), 15 deletions(-) diff --git a/README.md b/README.md index 668c32f..1b15c5f 100644 --- a/README.md +++ b/README.md @@ -41,8 +41,8 @@ Subsequent actions will have access to the following outputs: - The `directory` configuration that was used by dependabot for this updated Dependency. - `steps.dependabot-metadata.outputs.package-ecosystem` - The `package-ecosystem` configuration that was used by dependabot for this updated Dependency. -- `steps.dependabot-metadata.outputs.target_branch` - - The `target_branch` configuration that was used by dependabot for this updated Dependency. +- `steps.dependabot-metadata.outputs.target-branch` + - The `target-branch` configuration that was used by dependabot for this updated Dependency. **Note:** These outputs will only be populated if the target Pull Request was opened by Dependabot and contains **only** Dependabot-created commits. diff --git a/action.yml b/action.yml index 574a887..fe5bdd6 100644 --- a/action.yml +++ b/action.yml @@ -20,8 +20,8 @@ outputs: description: 'The `directory` configuration that was used by dependabot for this updated Dependency.' package-ecosystem: description: 'The `package-ecosystem` configuration that was used by dependabot for this updated Dependency.' - target_branch: - description: 'The `target_branch` configuration that was used by dependabot for this updated Dependency.' + target-branch: + description: 'The `target-branch` configuration that was used by dependabot for this updated Dependency.' runs: using: 'node12' main: 'dist/index.js' diff --git a/dist/index.js b/dist/index.js index fbdabbd..4fce7c3 100644 --- a/dist/index.js +++ b/dist/index.js @@ -13462,7 +13462,7 @@ function set(updatedDependencies) { core.info(`outputs.update-type: ${updateType}`); core.info(`outputs.directory: ${directory}`); core.info(`outputs.package-ecosystem: ${ecosystem}`); - core.info(`outputs.target_branch: ${target}`); + core.info(`outputs.target-branch: ${target}`); core.endGroup(); core.setOutput('updated-dependencies-json', updatedDependencies); core.setOutput('dependency-names', dependencyNames); @@ -13470,7 +13470,7 @@ function set(updatedDependencies) { core.setOutput('update-type', updateType); core.setOutput('directory', directory); core.setOutput('package-ecosystem', ecosystem); - core.setOutput('target_branch', target); + core.setOutput('target-branch', target); } function maxDependencyTypes(updatedDependencies) { const dependencyTypes = updatedDependencies.reduce(function (dependencyTypes, dependency) { @@ -13561,4 +13561,4 @@ run(); module.exports = __webpack_exports__; /******/ })() -; \ No newline at end of file +; diff --git a/src/dependabot/output.test.ts b/src/dependabot/output.test.ts index 5c8cd33..83e5ba2 100644 --- a/src/dependabot/output.test.ts +++ b/src/dependabot/output.test.ts @@ -34,7 +34,7 @@ test('when given a single dependency it sets its values', async () => { expect(core.setOutput).toBeCalledWith('update-type', 'version-update:semver-minor') expect(core.setOutput).toBeCalledWith('directory', 'wwwroot') expect(core.setOutput).toBeCalledWith('package-ecosystem', 'nuget') - expect(core.setOutput).toBeCalledWith('target_branch', 'main') + expect(core.setOutput).toBeCalledWith('target-branch', 'main') }) test('when given a multiple dependencies, it uses the highest values for types', async () => { @@ -82,7 +82,7 @@ test('when given a multiple dependencies, it uses the highest values for types', expect(core.setOutput).toBeCalledWith('update-type', 'version-update:semver-major') expect(core.setOutput).toBeCalledWith('directory', '') expect(core.setOutput).toBeCalledWith('package-ecosystem', '') - expect(core.setOutput).toBeCalledWith('target_branch', '') + expect(core.setOutput).toBeCalledWith('target-branch', '') }) test('when the dependency has no update type', async () => { @@ -110,7 +110,7 @@ test('when the dependency has no update type', async () => { expect(core.setOutput).toBeCalledWith('update-type', null) expect(core.setOutput).toBeCalledWith('directory', '') expect(core.setOutput).toBeCalledWith('package-ecosystem', '') - expect(core.setOutput).toBeCalledWith('target_branch', '') + expect(core.setOutput).toBeCalledWith('target-branch', '') }) test('when given a multiple dependencies, and some do not have update types', async () => { @@ -158,5 +158,5 @@ test('when given a multiple dependencies, and some do not have update types', as expect(core.setOutput).toBeCalledWith('update-type', 'version-update:semver-minor') expect(core.setOutput).toBeCalledWith('directory', '') expect(core.setOutput).toBeCalledWith('package-ecosystem', '') - expect(core.setOutput).toBeCalledWith('target_branch', '') + expect(core.setOutput).toBeCalledWith('target-branch', '') }) diff --git a/src/dependabot/output.ts b/src/dependabot/output.ts index 3a5671c..6f851d0 100644 --- a/src/dependabot/output.ts +++ b/src/dependabot/output.ts @@ -31,7 +31,7 @@ export function set (updatedDependencies: Array): void { core.info(`outputs.update-type: ${updateType}`) core.info(`outputs.directory: ${directory}`) core.info(`outputs.package-ecosystem: ${ecosystem}`) - core.info(`outputs.target_branch: ${target}`) + core.info(`outputs.target-branch: ${target}`) core.endGroup() core.setOutput('updated-dependencies-json', updatedDependencies) @@ -40,7 +40,7 @@ export function set (updatedDependencies: Array): void { core.setOutput('update-type', updateType) core.setOutput('directory', directory) core.setOutput('package-ecosystem', ecosystem) - core.setOutput('target_branch', target) + core.setOutput('target-branch', target) } function maxDependencyTypes (updatedDependencies: Array): string { diff --git a/src/main.test.ts b/src/main.test.ts index d6f40e9..eb9c4c4 100644 --- a/src/main.test.ts +++ b/src/main.test.ts @@ -102,7 +102,7 @@ test('it sets the updated dependency as an output for subsequent actions', async expect(core.setOutput).toBeCalledWith('update-type', 'version-update:semver-minor') expect(core.setOutput).toBeCalledWith('directory', '/') expect(core.setOutput).toBeCalledWith('package-ecosystem', 'nuget') - expect(core.setOutput).toBeCalledWith('target_branch', 'main') + expect(core.setOutput).toBeCalledWith('target-branch', 'main') }) test('if there are multiple dependencies, it summarizes them', async () => { @@ -164,7 +164,7 @@ test('if there are multiple dependencies, it summarizes them', async () => { expect(core.setOutput).toBeCalledWith('update-type', 'version-update:semver-major') expect(core.setOutput).toBeCalledWith('directory', 'api/main') expect(core.setOutput).toBeCalledWith('package-ecosystem', 'npm_and_yarn') - expect(core.setOutput).toBeCalledWith('target_branch', 'trunk') + expect(core.setOutput).toBeCalledWith('target-branch', 'trunk') }) test('it sets the action to failed if there is an unexpected exception', async () => {