Skip to content

Commit

Permalink
Merge pull request #5 from sankichi92/change-input-name
Browse files Browse the repository at this point in the history
Change input name: `common-dependency-filepaths` => `common-dependency-paths`
  • Loading branch information
sankichi92 authored Jul 3, 2024
2 parents 0d84a1b + 439f436 commit 41bc98c
Show file tree
Hide file tree
Showing 6 changed files with 18 additions and 18 deletions.
2 changes: 1 addition & 1 deletion .github/workflows/ci.yml
Original file line number Diff line number Diff line change
Expand Up @@ -51,5 +51,5 @@ jobs:
uses: ./
with:
target-file: "*"
common-dependency-filepaths: |-
common-dependency-paths: |-
action.yml
8 changes: 4 additions & 4 deletions README.md
Original file line number Diff line number Diff line change
Expand Up @@ -15,11 +15,11 @@ This action works only with the `push` and `pull_request` events due to the need

**Required**. A filename that must be included in the output directories.

### `common-dependency-filepaths`
### `common-dependency-paths`

File paths that all directories including the `target-file` depend on.
Paths to files that all directories including the `target-file` depend on.
If any of these files are changed, all directories including the `target-file` are considered changed.
Separate file paths with a newline.
Separate paths with a newline.

## Outputs

Expand Down Expand Up @@ -50,7 +50,7 @@ jobs:
id: list-changed-directories
with:
target-file: Gemfile.lock
common-dependency-filepaths: |-
common-dependency-paths: |-
.github/workflows/ruby.yml
test:
Expand Down
6 changes: 3 additions & 3 deletions action.yml
Original file line number Diff line number Diff line change
Expand Up @@ -5,10 +5,10 @@ inputs:
target-file:
description: A filename that must be included in the output directories
required: true
common-dependency-filepaths:
common-dependency-paths:
description: >-
File paths that all directories including the target-file depend on. If any of these files are changed,
all directories including the target-file are considered changed. Separate file paths with a newline.
Paths to files that all directories including the target-file depend on. If any of these files are changed,
all directories including the target-file are considered changed.
outputs:
changed-directories:
Expand Down
8 changes: 4 additions & 4 deletions dist/index.js

Some generated files are not rendered by default. Learn more about how customized files appear on GitHub.

2 changes: 1 addition & 1 deletion dist/index.js.map

Large diffs are not rendered by default.

10 changes: 5 additions & 5 deletions src/main.ts
Original file line number Diff line number Diff line change
Expand Up @@ -28,14 +28,14 @@ export async function run() {
await gitFetch(baseSHA);
core.endGroup();

const commonDependencyFilepaths = core.getMultilineInput(
"common-dependency-filepaths",
const commonDependencyPaths = core.getMultilineInput(
"common-dependency-paths",
);
if (commonDependencyFilepaths.length > 0) {
core.startGroup("Checking diff for common dependency files");
if (commonDependencyPaths.length > 0) {
core.startGroup("Checking diff for common dependency paths");
const isCommonDependencyChanged = await gitDiffExists(
baseSHA,
commonDependencyFilepaths,
commonDependencyPaths,
);
core.endGroup();

Expand Down

0 comments on commit 41bc98c

Please sign in to comment.