diff --git a/README.md b/README.md index 668c32f0..1b15c5fc 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 574a887a..fe5bdd61 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 fbdabbd5..70c6fc6d 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) { diff --git a/src/dependabot/output.test.ts b/src/dependabot/output.test.ts index 5c8cd330..83e5ba2b 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 3a5671cd..6f851d09 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 d6f40e94..eb9c4c49 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 () => {