feat: Add support for non ascii filenames (#473)

* feat: Add support for non ascii filenames

Closes: #437

* Update action.yml

* Update README.md

* Create test-è.txt

* Update test.yml

* Update README.md

* Update action.yml

* Update entrypoint.sh

* Update entrypoint.sh

* Update entrypoint.sh

.github/workflows/codacy-analysis.yml

@@ -44,6 +44,6 @@ jobs:
 
       # Upload the SARIF file generated in the previous step
       - name: Upload SARIF results file
-        uses: github/codeql-action/upload-sarif@v1
+        uses: github/codeql-action/upload-sarif@v2
         with:
           sarif_file: results.sarif

.github/workflows/submodule-sync.yml

@@ -19,7 +19,7 @@ jobs:
           git submodule update --remote --recursive
       
       - name: Create Pull Request
-        uses: peter-evans/create-pull-request@v4.0.0
+        uses: peter-evans/create-pull-request@v4.0.2
         with:
           title: "Updated submodule"
           branch: "chore/update-submodule"

.github/workflows/sync-release-version.yml

@@ -8,7 +8,7 @@ jobs:
   update-version:
     runs-on: ubuntu-latest
     steps:
-      - uses: actions/checkout@v3.0.0
+      - uses: actions/checkout@v3.0.2
         with:
           fetch-depth: 0
       - name: Sync release version.
@@ -23,7 +23,7 @@ jobs:
         with:
           output: 'HISTORY.md'
       - name: Create Pull Request
-        uses: peter-evans/create-pull-request@v4.0.0
+        uses: peter-evans/create-pull-request@v4.0.2
         with:
           base: "main"
           title: "Upgraded to ${{ steps.sync-release-version.outputs.new_version }}"

.github/workflows/test.yml

@@ -17,7 +17,7 @@ jobs:
       - name: Checkout to branch
         uses: actions/checkout@v3
       - name: shellcheck
-        uses: reviewdog/action-shellcheck@v1.14
+        uses: reviewdog/action-shellcheck@v1.15
   
   test-pull-requests-from-forks:
     name: Test changed-files diff on pull_requests from forks
@@ -231,6 +231,16 @@ jobs:
           echo "Your README.md has been modified ${{ steps.changed-files.outputs.modified_files }}."
         shell:
           bash
+      - name: Run changed-files with quotepath disabled
+        id: changed-files-quotepath
+        uses: ./
+        with:
+          quotepath: "false"
+      - name: Show output
+        run: |
+          echo '${{ toJSON(steps.changed-files-quotepath.outputs) }}'
+        shell:
+          bash
       - name: Run changed-files with forward slash separator
         id: changed-files-forward-slash
         uses: ./

.github/workflows/update-readme.yml

@@ -9,7 +9,7 @@ jobs:
   sync-assets:
     runs-on: ubuntu-latest
     steps:
-      - uses: actions/checkout@v3.0.0
+      - uses: actions/checkout@v3.0.2
         with:
           fetch-depth: 0
 
@@ -31,7 +31,7 @@ jobs:
 
       - name: Create Pull Request
         if: failure()
-        uses: peter-evans/create-pull-request@v4.0.0
+        uses: peter-evans/create-pull-request@v4.0.2
         with:
           base: "main"
           title: "Updated README.md"

HISTORY.md

@@ -1,5 +1,38 @@
 # Changelog
 
+## [v18.7](https://github.com/tj-actions/changed-files/tree/v18.7) (2022-04-08)
+
+[Full Changelog](https://github.com/tj-actions/changed-files/compare/v18.6...v18.7)
+
+**Fixed bugs:**
+
+- \[BUG\] Modified files treated as `Non Matching modified files` [\#450](https://github.com/tj-actions/changed-files/issues/450)
+
+**Closed issues:**
+
+- Dependency Dashboard [\#27](https://github.com/tj-actions/changed-files/issues/27)
+
+**Merged pull requests:**
+
+- chore\(deps\): update peter-evans/create-pull-request action to v4.0.1 [\#461](https://github.com/tj-actions/changed-files/pull/461) ([renovate[bot]](https://github.com/apps/renovate))
+- Upgraded to v18.6 [\#460](https://github.com/tj-actions/changed-files/pull/460) ([jackton1](https://github.com/jackton1))
+
+## [v18.6](https://github.com/tj-actions/changed-files/tree/v18.6) (2022-03-30)
+
+[Full Changelog](https://github.com/tj-actions/changed-files/compare/v18.5...v18.6)
+
+**Fixed bugs:**
+
+- \[BUG\] Intermittent "Unable to locate the current sha" [\#458](https://github.com/tj-actions/changed-files/issues/458)
+
+**Merged pull requests:**
+
+- fix: resolved error with escaping unicode unsafe characters [\#459](https://github.com/tj-actions/changed-files/pull/459) ([jackton1](https://github.com/jackton1))
+- chore: remove unused code [\#457](https://github.com/tj-actions/changed-files/pull/457) ([jackton1](https://github.com/jackton1))
+- chore: test changes to .github workflows files [\#456](https://github.com/tj-actions/changed-files/pull/456) ([jackton1](https://github.com/jackton1))
+- chore: test filenames that should be escaped [\#455](https://github.com/tj-actions/changed-files/pull/455) ([jackton1](https://github.com/jackton1))
+- Upgraded to v18.5 [\#454](https://github.com/tj-actions/changed-files/pull/454) ([jackton1](https://github.com/jackton1))
+
 ## [v18.5](https://github.com/tj-actions/changed-files/tree/v18.5) (2022-03-29)
 
 [Full Changelog](https://github.com/tj-actions/changed-files/compare/v18.4...v18.5)
@@ -10,13 +43,8 @@
 
 **Fixed bugs:**
 
-- \[BUG\] Modified files treated as `Non Matching modified files` [\#450](https://github.com/tj-actions/changed-files/issues/450)
 - \[BUG\] Not able to compare current commit with the specific commit of a branch\(in the Pull request event\) [\#441](https://github.com/tj-actions/changed-files/issues/441)
 
-**Closed issues:**
-
-- Dependency Dashboard [\#27](https://github.com/tj-actions/changed-files/issues/27)
-
 **Merged pull requests:**
 
 - fix: bug passing invalid patterns to grep [\#453](https://github.com/tj-actions/changed-files/pull/453) ([jackton1](https://github.com/jackton1))

README.md

@@ -68,7 +68,7 @@ jobs:
 
       - name: Get changed files
         id: changed-files
-        uses: tj-actions/changed-files@v18.5
+        uses: tj-actions/changed-files@v18.7
 
       - name: List all changed files
         run: |
@@ -140,6 +140,7 @@ Support this project with a :star:
 |               path                |        `string`        | `false`  |                       |                                                                                                                              Relative path under <br> `GITHUB_WORKSPACE` <br> to the repository                                                                                                                               |
 |    since\_last\_remote\_commit    |        `string`        | `false`  |        `false`        |                                                                 Use the last commit on the remote <br> branch as the `base_sha` <br> (Defaults to the last commit <br> on the target branch for Pull requests <br> or the previous commit <br> on the current branch <br> for push events). <br /> NOTE: This requires <br /> `fetch-depth: 0` <br /> with `actions/checkout@v2`                                                                  |
 |          use\_fork\_point           |        `string`        | `false`  |        `false`        |             Finds best common ancestor <br /> between two commits <br /> to use in a three-way merge <br /> as the `base_sha` <br /> See: [git merge-base](https://git-scm.com/docs/git-merge-base#Documentation/git-merge-base.txt---fork-point). <br> NOTE: This pulls the entire commit history of the base branch              |
+|    quotepath                        |        `string`       |   `false` |        `true`     | Output filenames completely verbatim by setting this to `false` |
 
 ## Example
 
@@ -153,11 +154,11 @@ Support this project with a :star:
 
       - name: Get changed files using defaults
         id: changed-files
-        uses: tj-actions/changed-files@v18.5
+        uses: tj-actions/changed-files@v18.7
 
       - name: Get changed files using a comma separator
         id: changed-files-comma
-        uses: tj-actions/changed-files@v18.5
+        uses: tj-actions/changed-files@v18.7
         with:
           separator: ","
 
@@ -179,7 +180,7 @@ Support this project with a :star:
 
       - name: Get specific changed files
         id: changed-files-specific
-        uses: tj-actions/changed-files@v18.5
+        uses: tj-actions/changed-files@v18.7
         with:
           files: |
             my-file.txt
@@ -220,14 +221,14 @@ Support this project with a :star:
 
       - name: Use a source file or list of file(s) to populate to files input.
         id: changed-files-specific-source-file
-        uses: tj-actions/changed-files@v18.5
+        uses: tj-actions/changed-files@v18.7
         with:
           files_from_source_file: |
             test/changed-files-list.txt
 
       - name: Use a source file or list of file(s) to populate to files input and optionally specify more files.
         id: changed-files-specific-source-file-and-specify-files
-        uses: tj-actions/changed-files@v18.5
+        uses: tj-actions/changed-files@v18.7
         with:
           files_from_source_file: |
             test/changed-files-list.txt
@@ -236,13 +237,13 @@ Support this project with a :star:
 
       - name: Use a different commit SHA
         id: changed-files-custom-sha
-        uses: tj-actions/changed-files@v18.5
+        uses: tj-actions/changed-files@v18.7
         with:
           sha: ${{ github.event.pull_request.head.sha }}
 
       - name: Use a different base SHA
         id: changed-files-custom-base-sha
-        uses: tj-actions/changed-files@v18.5
+        uses: tj-actions/changed-files@v18.7
         with:
           base_sha: "2096ed0"
           
@@ -254,7 +255,7 @@ Support this project with a :star:
 
       - name: Run changed-files with defaults on the dir1
         id: changed-files-for-dir1
-        uses: tj-actions/changed-files@v18.5
+        uses: tj-actions/changed-files@v18.7
         with:
           path: dir1
 
@@ -266,13 +267,13 @@ Support this project with a :star:
 
       - name: Run changed-files using the last commit on the remote branch
         id: changed-files-since-last-remote-commit
-        uses: tj-actions/changed-files@v18.5
+        uses: tj-actions/changed-files@v18.7
         with:
           since_last_remote_commit: "true"
       
       - name: Run changed-files using the fork point of a pull request
         id: changed-files-fork-point
-        uses: tj-actions/changed-files@v18.5
+        uses: tj-actions/changed-files@v18.7
         with:
           use_fork_point: "true"
 
@@ -288,7 +289,7 @@ Support this project with a :star:
 >
 > *   Using characters like `\n`, `%` and `\r` as separators would be [URL encoded](https://www.w3schools.com/tags/ref_urlencode.asp)
 > *   Spaces in file names can introduce bugs when using bash loops. See: [#216](https://github.com/tj-actions/changed-files/issues/216)
-> *   However, this action will handle spaces in file names, with a recommendation of using a separator to prevent hidden issues.
+>     However, this action will handle spaces in file names, with a recommendation of using a separator to prevent hidden issues.
 >     ![Screen Shot 2021-10-23 at 9 37 34 AM](https://user-images.githubusercontent.com/17484350/138558767-b13c90bf-a1ae-4e86-9520-70a6a4624f41.png)
 
 ## Versioning
@@ -326,7 +327,7 @@ With the switch from using grep's Extended regex to match files to the natively
 
       - name: Get specific changed files
         id: changed-files-specific
-        uses: tj-actions/changed-files@v18.5
+        uses: tj-actions/changed-files@v18.7
         with:
           files: |
             *.sh

action.yml

@@ -1,5 +1,5 @@
-name: Changed files
-description: Get all changed files for push and pull request events.
+name: Changed Files
+description: Get all Added, Copied, Modified, Deleted, Renamed, Type changed, Unmerged, Unknown files.
 author: tj-actions
 
 inputs:
@@ -53,6 +53,10 @@ inputs:
     description: 'Finds best common ancestor between two commits to use in a three-way merge as the base_sha'
     default: 'false'
     required: false
+  quotepath:
+    description: 'Output filenames completely verbatim by setting this to false'
+    default: 'true'
+    required: false
 
 outputs:
   added_files:
@@ -147,7 +151,7 @@ runs:
         INPUT_PATH: ${{ inputs.path }}
         INPUT_USE_FORK_POINT: ${{ inputs.use_fork_point }}
     - name: Glob match
-      uses: tj-actions/glob@v7.15
+      uses: tj-actions/glob@v7.16
       id: glob
       with:
         files: ${{ inputs.files }}
@@ -177,6 +181,7 @@ runs:
         INPUT_CURRENT_SHA: ${{ steps.changed-files-diff-sha.outputs.current_sha }}
         INPUT_TARGET_BRANCH: ${{ steps.changed-files-diff-sha.outputs.target_branch }}
         INPUT_CURRENT_BRANCH: ${{ steps.changed-files-diff-sha.outputs.current_branch }}
+        INPUT_QUOTEPATH: ${{ inputs.quotepath }}
 
 branding:
   icon: file-text

entrypoint.sh

@@ -6,6 +6,12 @@ INPUT_SEPARATOR="${INPUT_SEPARATOR//'%'/'%25'}"
 INPUT_SEPARATOR="${INPUT_SEPARATOR//$'\n'/'%0A'}"
 INPUT_SEPARATOR="${INPUT_SEPARATOR//$'\r'/'%0D'}"
 
+if [[ $INPUT_QUOTEPATH == "false" ]]; then
+  git config --global core.quotepath off
+else
+  git config --global core.quotepath on
+fi
+
 function get_diff() {
   base="$1"
   sha="$2"

test/test-è.txt

@@ -0,0 +1 @@
+This is a test file.