Skip to content

Commit

Permalink
remove: deprecated dir_names_exclude_root (#1291)
Browse files Browse the repository at this point in the history
Co-authored-by: repo-ranger[bot] <39074581+repo-ranger[bot]@users.noreply.github.com>
  • Loading branch information
jackton1 and repo-ranger[bot] committed Jun 23, 2023
1 parent 63f203d commit faa5db4
Show file tree
Hide file tree
Showing 6 changed files with 7 additions and 23 deletions.
4 changes: 2 additions & 2 deletions .github/workflows/test.yml
Original file line number Diff line number Diff line change
Expand Up @@ -920,15 +920,15 @@ jobs:
exit 1
shell:
bash
- name: Run changed-files with dir_names and dir_names_exclude_root
- name: Run changed-files with dir_names and dir_names_exclude_current_dir
id: changed-files-dir-names-exclude-root
uses: ./
with:
base_sha: dddfbd69
sha: ce8c1983
fetch_depth: 60000
dir_names: "true"
dir_names_exclude_root: "true"
dir_names_exclude_current_dir: "true"
dir_names_max_depth: "1"
- name: Show output
run: |
Expand Down
5 changes: 0 additions & 5 deletions action.yml
Original file line number Diff line number Diff line change
Expand Up @@ -112,11 +112,6 @@ inputs:
description: "Exclude the current directory represented by `.` from the output when `dir_names` is set to `true`."
required: false
default: "false"
dir_names_exclude_root:
description: "Exclude the root directory represented by `.` from the output when `dir_names`is set to `true`."
required: false
default: "false"
deprecationMessage: "Use `dir_names_exclude_current_dir` instead."
json:
description: "Output list of changed files in a JSON formatted string which can be used for matrix jobs."
required: false
Expand Down
8 changes: 2 additions & 6 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.

6 changes: 2 additions & 4 deletions src/changedFiles.ts
Original file line number Diff line number Diff line change
Expand Up @@ -172,8 +172,7 @@ function* getChangeTypeFilesGenerator({
yield getDirnameMaxDepth({
pathStr: file,
dirNamesMaxDepth: inputs.dirNamesMaxDepth,
excludeCurrentDir:
inputs.dirNamesExcludeRoot || inputs.dirNamesExcludeCurrentDir
excludeCurrentDir: inputs.dirNamesExcludeCurrentDir
})
} else {
yield file
Expand Down Expand Up @@ -220,8 +219,7 @@ function* getAllChangeTypeFilesGenerator({
yield getDirnameMaxDepth({
pathStr: file,
dirNamesMaxDepth: inputs.dirNamesMaxDepth,
excludeCurrentDir:
inputs.dirNamesExcludeRoot || inputs.dirNamesExcludeCurrentDir
excludeCurrentDir: inputs.dirNamesExcludeCurrentDir
})
} else {
yield file
Expand Down
5 changes: 0 additions & 5 deletions src/inputs.ts
Original file line number Diff line number Diff line change
Expand Up @@ -28,7 +28,6 @@ export type Inputs = {
diffRelative: boolean
dirNames: boolean
dirNamesMaxDepth?: number
dirNamesExcludeRoot: boolean
dirNamesExcludeCurrentDir: boolean
json: boolean
escapeJson: boolean
Expand Down Expand Up @@ -125,9 +124,6 @@ export const getInputs = (): Inputs => {
const dirNamesMaxDepth = core.getInput('dir_names_max_depth', {
required: false
})
const dirNamesExcludeRoot = core.getBooleanInput('dir_names_exclude_root', {
required: false
})
const dirNamesExcludeCurrentDir = core.getBooleanInput(
'dir_names_exclude_current_dir',
{
Expand Down Expand Up @@ -191,7 +187,6 @@ export const getInputs = (): Inputs => {
oldNewFilesSeparator,
// End Not Supported via REST API
dirNames,
dirNamesExcludeRoot,
dirNamesExcludeCurrentDir,
json,
escapeJson,
Expand Down

0 comments on commit faa5db4

Please sign in to comment.