PR for testing solutions for issues accessing TS action (#4)

* use action path

* try self-referencing

* Update action.yml

* Update action.yml

* Update action.yml

* rm glob pattern

* Update action.yml

* Update action.yml

* Update action.yml

* Update action.yml

* Update action.yml

* remove composite action

* update README.md

* Fix typo

Author: nedredmond

.github/workflows/ci.yml

@@ -45,10 +45,23 @@ jobs:
     runs-on: ubuntu-latest
 
     steps:
-      - name: Checkout
-        id: checkout
+      - name: Checkout repository
         uses: actions/checkout@v4
+        with:
+          fetch-depth: 0
+
+      - name: Get changed files
+        uses: Khan/actions@get-changed-files-v1
+        id: changed
 
-      - name: Test Local Action
-        id: test-action
+      - name: Filter out files that don't need a changeset
+        uses: Khan/actions@filter-files-v0
+        id: match
+        with:
+          changed-files: ${{ steps.changed.outputs.files }}
+          invert: true
+
+      - name: Verify changeset entries
         uses: ./
+        with:
+          changed_files: ${{ steps.match.outputs.filtered }}

README.md

@@ -5,14 +5,9 @@
 ![CI](https://github.com/actions/typescript-action/actions/workflows/ci.yml/badge.svg)
 
 This action validates that there is a changeset entry for each package that has
-been changed.
-
-This repository has two action files:
-
-| File                                                                                 | Description                                                                                                                                                                               |
-| ------------------------------------------------------------------------------------ | ----------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------- |
-| [`./action.yml`](./action.yml)                                                       | This composite action calls on other Khan actions to prepare a list of changed files.                                                                                                     |
-| [`./actions/verify-per-package/action.yml`](./actions/verify-per-package/action.yml) | This ts action contains its logic to [`./src/main.ts`](./src/main.ts). It checks for packages corresponding to changed files, then determines if there is a match for each in changesets. |
+been changed for a given list of files. It is intended to be used in a workflow
+with other actions from the [Khan/actions](https://github.com/Khan/actions)
+repo.
 
 ## When Making Changes
 
@@ -41,5 +36,27 @@ steps:
 
   - name: Verify Changeset Per Package
     id: changeset-per-package
-    uses: Khan/changeset-per-package@v1 # Commit with the `v1` tag
+    uses: Khan/[email protected]
+
+steps:
+  - name: Checkout with history
+    uses: actions/checkout@v4
+    with:
+      fetch-depth: 0 # This is necessary! Checks out the history so that we can
+                     # get the changset entries diff.
+
+  - name: Get changed files
+    uses: Khan/actions@get-changed-files-v1
+    id: changed
+
+  - name: Filter out files that don't need changeset
+    uses: Khan/actions@filter-files-v0
+    id: match
+    with:
+      changed-files: ${{ steps.changed.outputs.files }}
+
+  - name: Verify changeset entries
+    uses: Khan/[email protected]
+    with:
+      changed_files: ${{ steps.match.outputs.filtered }}
 ```

action.yml

@@ -3,48 +3,10 @@ description: 'Fail if there is not a valid changeset entry for each package.'
 author: 'Ned Redmond'
 
 inputs:
-  exclude_files_and_dirs:
-    description:
-      list of comma-separated paths (files or directories ending in /) that
-      don't need a changeset when changed. Default .github/
-    required: false
-    default: .github/
-  exclude_extensions:
-    description:
-      a list of comma-separated extensions that don't need a changeset when
-      changed. Default <empty>
-    required: false
-    default: ''
-  exclude_globs:
-    description:
-      a list of comma-separated globs (using picomatch syntax) that don't need a
-      changeset when changed. Default <empty>
-    required: false
-    default: ''
+  changed_files:
+    description: list of changed files
+    required: true
 
 runs:
-  using: 'composite'
-  steps:
-    - name: Checkout repository
-      uses: actions/checkout@v4
-      with:
-        fetch-depth: 0
-
-    - name: Get changed files
-      uses: Khan/actions@get-changed-files-v1
-      id: changed
-
-    - name: Filter out files that don't need a changeset
-      uses: Khan/actions@filter-files-v0
-      id: match
-      with:
-        changed-files: ${{ steps.changed.outputs.files }}
-        invert: true
-        files: ${{ inputs.exclude_files_and_dirs }}
-        extensions: ${{ inputs.exclude_extensions }}
-        globs: ${{ inputs.exclude_globs }}
-
-    - name: Veryify changeset entries
-      uses: ./actions/verify-per-package
-      with:
-        changed_files: ${{ steps.match.outputs.filtered }}
+  using: node20
+  main: dist/index.js

actions/verify-per-package/action.yml

@@ -29,7 +29,7 @@
     "format:write": "prettier --write **/*.ts",
     "format:check": "prettier --check **/*.ts",
     "lint": "npx eslint . -c ./.github/linters/.eslintrc.yml",
-    "package": "ncc build src/index.ts -o actions/verify-per-package/dist --license licenses.txt",
+    "package": "ncc build src/index.ts --license licenses.txt",
     "package:watch": "yarn package -- --watch",
     "test": "(jest && make-coverage-badge --output-path ./badges/coverage.svg) || make-coverage-badge --output-path ./badges/coverage.svg",
     "all": "yarn format:write && yarn lint && yarn test && yarn package"

actions/verify-per-package/dist/index.js dist/index.js

@@ -7,13 +7,13 @@
     "moduleResolution": "NodeNext",
     "baseUrl": "./",
     "sourceMap": true,
-    "outDir": "./dist",
+    "outDir": "dist",
     "noImplicitAny": true,
     "esModuleInterop": true,
     "forceConsistentCasingInFileNames": true,
     "strict": true,
     "skipLibCheck": true,
     "newLine": "lf"
   },
-  "exclude": ["./dist", "./node_modules", "./__tests__", "./coverage"]
+  "exclude": ["dist", "./node_modules", "./__tests__", "./coverage"]
 }