- 
                Notifications
    You must be signed in to change notification settings 
- Fork 6
Open
Description
The default for actions/setup-node@v1 is node 10, which does not work with this action. I saw that the action itself is configured to run using node 12, but the cache-restore and cache-save scripts just use the node that's on PATH, which causes a silent failure to restore/save the cache:
  (node:6173) UnhandledPromiseRejectionWarning: TypeError: Object.fromEntries is not a function
      at Object.<anonymous> (/Users/runner/work/_actions/ktmud/cached-dependencies/v1/dist/scripts/cache/index.js:5812:39)
      at Generator.next (<anonymous>)
      at /Users/runner/work/_actions/ktmud/cached-dependencies/v1/dist/scripts/cache/index.js:5763:71
      at new Promise (<anonymous>)
      at module.exports.935.__awaiter (/Users/runner/work/_actions/ktmud/cached-dependencies/v1/dist/scripts/cache/index.js:5759:12)
      at Object.applyInputs (/Users/runner/work/_actions/ktmud/cached-dependencies/v1/dist/scripts/cache/index.js:5811:12)
      at Object.save (/Users/runner/work/_actions/ktmud/cached-dependencies/v1/dist/scripts/cache/index.js:5180:25)
      at Object.<anonymous> (/Users/runner/work/_actions/ktmud/cached-dependencies/v1/dist/scripts/cache/index.js:5204:46)
      at Generator.next (<anonymous>)
      at fulfilled (/Users/runner/work/_actions/ktmud/cached-dependencies/v1/dist/scripts/cache/index.js:5048:58)
If it's possible to make the cache-restore and cache-save scripts use node 12, that'd be great. If not, it would be good to at least document the requirement to run setup-node with node-version: '12'.
Metadata
Metadata
Assignees
Labels
No labels