chore: remove upstream CI workflows (mirror inertization per internal#233)

.github/workflows/build.yml

@@ -1,32 +0,0 @@
-name: "Build"
-
-on:
-  push:
-    paths-ignore: [ '*.md' ]
-    branches:
-      - master
-
-jobs:
-  build:
-    runs-on: ubuntu-latest
-    steps:
-    - uses: actions/checkout@v6
-    - uses: actions/setup-node@v6
-      with:
-        node-version: 24
-        cache: 'npm'
-    - run: |
-        npm install
-        npm run all
-
-  self-test:
-    runs-on: ubuntu-latest
-    steps:
-    - uses: actions/checkout@v6
-    - uses: ./
-      id: filter
-      with:
-        filters: '.github/filters.yml'
-    - name: filter-test
-      if: steps.filter.outputs.any != 'true' || steps.filter.outputs.error == 'true'
-      run: exit 1

.github/workflows/pull-request-verification.yml

@@ -1,152 +0,0 @@
-name: "Pull Request Verification"
-on:
-  pull_request:
-    paths-ignore: [ '*.md' ]
-    branches:
-      - master
-      - '**'
-
-jobs:
-  build:
-    runs-on: ubuntu-latest
-    steps:
-    - uses: actions/checkout@v6
-    - uses: actions/setup-node@v6
-      with:
-        node-version: 24
-        cache: 'npm'
-    - run: |
-        npm install
-        npm run all
-    - name: Check dist is up to date
-      run: |
-        if [ -n "$(git diff --name-only dist/)" ]; then
-          echo "::error::dist/index.js is out of date. Run 'npm run all' and commit the result."
-          git diff --stat dist/
-          exit 1
-        fi
-
-  test-inline:
-    runs-on: ubuntu-latest
-    permissions:
-      pull-requests: read
-    steps:
-    - uses: actions/checkout@v6
-    - uses: ./
-      id: filter
-      with:
-        filters: |
-          error:
-            - not_existing_path/**/*
-          any:
-            - "**/*"
-    - name: filter-test
-      if: steps.filter.outputs.any != 'true' || steps.filter.outputs.error == 'true'
-      run: exit 1
-    - name: changes-test
-      if: contains(fromJSON(steps.filter.outputs.changes), 'error') || !contains(fromJSON(steps.filter.outputs.changes), 'any')
-      run: exit 1
-
-  test-external:
-    runs-on: ubuntu-latest
-    permissions:
-      pull-requests: read
-    steps:
-    - uses: actions/checkout@v6
-    - uses: ./
-      id: filter
-      with:
-        filters: '.github/filters.yml'
-    - name: filter-test
-      if: steps.filter.outputs.any != 'true' || steps.filter.outputs.error == 'true'
-      run: exit 1
-
-  test-without-token:
-    runs-on: ubuntu-latest
-    steps:
-    - uses: actions/checkout@v6
-    - uses: ./
-      id: filter
-      with:
-        token: ''
-        filters: '.github/filters.yml'
-    - name: filter-test
-      if: steps.filter.outputs.any != 'true' || steps.filter.outputs.error == 'true'
-      run: exit 1
-
-  test-wd-without-token:
-    runs-on: ubuntu-latest
-    steps:
-    - uses: actions/checkout@v6
-      with:
-        path: somewhere
-    - uses: ./somewhere
-      id: filter
-      with:
-        token: ''
-        working-directory: somewhere
-        filters: '.github/filters.yml'
-    - name: filter-test
-      if: steps.filter.outputs.any != 'true' || steps.filter.outputs.error == 'true'
-      run: exit 1
-
-  test-local-changes:
-    runs-on: ubuntu-latest
-    steps:
-    - uses: actions/checkout@v6
-    - run: echo "NEW FILE" > local
-    - run: git add local
-    - uses: ./
-      id: filter
-      with:
-        base: HEAD
-        filters: |
-          local:
-            - local
-    - name: filter-test
-      if: steps.filter.outputs.local != 'true'
-      run: exit 1
-    - name: count-test
-      if: steps.filter.outputs.local_count != 1
-      run: exit 1
-
-  test-change-type:
-    runs-on: ubuntu-latest
-    steps:
-    - uses: actions/checkout@v6
-    - name: configure GIT user
-      run: git config user.email "john@nowhere.local" && git config user.name "John Doe"
-    - name: modify working tree
-      run: touch add.txt && rm README.md && echo "TEST" > LICENSE
-    - name: commit changes
-      run: git add -A && git commit -a -m 'testing this action'
-    - uses: ./
-      id: filter
-      with:
-        token: ''
-        list-files: shell
-        filters: |
-          added:
-            - added: "add.txt"
-          deleted:
-            - deleted: "README.md"
-          modified:
-            - modified: "LICENSE"
-          any:
-            - added|deleted|modified: "*"
-    - name: Print 'added_files'
-      run: echo ${{steps.filter.outputs.added_files}}
-    - name: Print 'modified_files'
-      run: echo ${{steps.filter.outputs.modified_files}}
-    - name: Print 'deleted_files'
-      run: echo ${{steps.filter.outputs.deleted_files}}
-    - name: filter-test
-      if: |
-        steps.filter.outputs.added != 'true'
-        || steps.filter.outputs.deleted != 'true'
-        || steps.filter.outputs.modified != 'true'
-        || steps.filter.outputs.any != 'true'
-        || steps.filter.outputs.added_files != 'add.txt'
-        || steps.filter.outputs.modified_files != 'LICENSE'
-        || steps.filter.outputs.deleted_files != 'README.md'
-      run: exit 1

CHANGELOG.md

@@ -1,8 +1,5 @@
 # Changelog
 
-## v4.1.0
-- [Allow base/ref override on pull_request events](https://github.com/dorny/paths-filter/pull/NNN)
-
 ## v4.0.0
 - [Update action runtime to node24](https://github.com/dorny/paths-filter/pull/294)
 

README.md

@@ -117,7 +117,7 @@ For more information, see [CHANGELOG](https://github.com/dorny/paths-filter/blob
     # introduced by the current branch are considered.
     # All files are considered as added if there is no common ancestor with
     # base branch or no previous commit.
-    # This option is ignored if action is triggered by pull_request event, unless 'allow-override-on-pr' is set to true.
+    # This option is ignored if action is triggered by pull_request event.
     # Default: repository default branch (e.g. master)
     base: ''
 
@@ -126,7 +126,7 @@ For more information, see [CHANGELOG](https://github.com/dorny/paths-filter/blob
     # but you want to get changes on a different branch.
     # If this is empty and action is triggered by merge_group event,
     # the head commit in the event will be used.
-    # This option is ignored if action is triggered by pull_request event, unless 'allow-override-on-pr' is set to true.
+    # This option is ignored if action is triggered by pull_request event.
     # default: ${{ github.ref }}
     ref:
 
@@ -178,14 +178,6 @@ For more information, see [CHANGELOG](https://github.com/dorny/paths-filter/blob
     #  - '!**/*.jpeg'
     #  - '!**/*.md'
     predicate-quantifier: 'some'
-
-    # When true, the user-provided `base` and/or `ref` inputs are honored even if
-    # the action is triggered by a pull_request, pull_request_review,
-    # pull_request_review_comment, or pull_request_target event. In that case the
-    # action skips the GitHub API path and uses git diff against the provided
-    # base/ref. Has no effect if `base` and `ref` are both empty.
-    # Default: false
-    allow-override-on-pr: 'false'
 ```
 
 ## Outputs

action.yml

@@ -49,14 +49,6 @@ inputs:
       allows to override the "at least one pattern" behavior to make it so that all of the patterns have to match or otherwise the file is excluded.
     required: false
     default: 'some'
-  allow-override-on-pr:
-    description: |
-      When true, the user-provided `base` and/or `ref` inputs are honored even if the action
-      is triggered by a pull_request, pull_request_review, pull_request_review_comment, or
-      pull_request_target event. In that case the action skips the GitHub API path and uses
-      git diff against the provided base/ref. Has no effect if `base` and `ref` are both empty.
-    required: false
-    default: 'false'
 outputs:
   changes:
     description: JSON array with names of all filters matching any of changed files

dist/index.js

@@ -575,7 +575,6 @@ async function run() {
         const listFiles = core.getInput('list-files', { required: false }).toLowerCase() || 'none';
         const initialFetchDepth = parseInt(core.getInput('initial-fetch-depth', { required: false })) || 10;
         const predicateQuantifier = core.getInput('predicate-quantifier', { required: false }) || filter_1.PredicateQuantifier.SOME;
-        const allowOverrideOnPr = core.getBooleanInput('allow-override-on-pr', { required: false });
         if (!isExportFormat(listFiles)) {
             core.setFailed(`Input parameter 'list-files' is set to invalid value '${listFiles}'`);
             return;
@@ -587,7 +586,7 @@ async function run() {
         }
         const filterConfig = { predicateQuantifier };
         const filter = new filter_1.Filter(filtersYaml, filterConfig);
-        const files = await getChangedFiles(token, base, ref, initialFetchDepth, allowOverrideOnPr);
+        const files = await getChangedFiles(token, base, ref, initialFetchDepth);
         core.info(`Detected ${files.length} changed files`);
         const results = filter.match(files);
         exportResults(results, listFiles);
@@ -608,7 +607,7 @@ function getConfigFileContent(configPath) {
     }
     return fs.readFileSync(configPath, { encoding: 'utf8' });
 }
-async function getChangedFiles(token, base, ref, initialFetchDepth, allowOverrideOnPr) {
+async function getChangedFiles(token, base, ref, initialFetchDepth) {
     var _a, _b;
     // if base is 'HEAD' only local uncommitted changes will be detected
     // This is the simplest case as we don't need to fetch more commits or evaluate current/before refs
@@ -625,10 +624,6 @@ async function getChangedFiles(token, base, ref, initialFetchDepth, allowOverrid
         case 'pull_request_review':
         case 'pull_request_review_comment':
         case 'pull_request_target': {
-            if (allowOverrideOnPr && (base || ref)) {
-                core.info(`'allow-override-on-pr' is enabled and base/ref were provided — skipping PR API and using git diff`);
-                return getChangedFilesFromGit(base, ref, initialFetchDepth);
-            }
             if (ref) {
                 core.warning(`'ref' input parameter is ignored when action is triggered by pull request event`);
             }

src/main.ts

@@ -34,7 +34,6 @@ async function run(): Promise<void> {
     const listFiles = core.getInput('list-files', {required: false}).toLowerCase() || 'none'
     const initialFetchDepth = parseInt(core.getInput('initial-fetch-depth', {required: false})) || 10
     const predicateQuantifier = core.getInput('predicate-quantifier', {required: false}) || PredicateQuantifier.SOME
-    const allowOverrideOnPr = core.getBooleanInput('allow-override-on-pr', {required: false})
 
     if (!isExportFormat(listFiles)) {
       core.setFailed(`Input parameter 'list-files' is set to invalid value '${listFiles}'`)
@@ -50,7 +49,7 @@ async function run(): Promise<void> {
     const filterConfig: FilterConfig = {predicateQuantifier}
 
     const filter = new Filter(filtersYaml, filterConfig)
-    const files = await getChangedFiles(token, base, ref, initialFetchDepth, allowOverrideOnPr)
+    const files = await getChangedFiles(token, base, ref, initialFetchDepth)
     core.info(`Detected ${files.length} changed files`)
     const results = filter.match(files)
     exportResults(results, listFiles)
@@ -75,13 +74,7 @@ function getConfigFileContent(configPath: string): string {
   return fs.readFileSync(configPath, {encoding: 'utf8'})
 }
 
-async function getChangedFiles(
-  token: string,
-  base: string,
-  ref: string,
-  initialFetchDepth: number,
-  allowOverrideOnPr: boolean
-): Promise<File[]> {
+async function getChangedFiles(token: string, base: string, ref: string, initialFetchDepth: number): Promise<File[]> {
   // if base is 'HEAD' only local uncommitted changes will be detected
   // This is the simplest case as we don't need to fetch more commits or evaluate current/before refs
   if (base === git.HEAD) {
@@ -98,10 +91,6 @@ async function getChangedFiles(
     case 'pull_request_review':
     case 'pull_request_review_comment':
     case 'pull_request_target': {
-      if (allowOverrideOnPr && (base || ref)) {
-        core.info(`'allow-override-on-pr' is enabled and base/ref were provided — skipping PR API and using git diff`)
-        return getChangedFilesFromGit(base, ref, initialFetchDepth)
-      }
       if (ref) {
         core.warning(`'ref' input parameter is ignored when action is triggered by pull request event`)
       }