action-detect-and-tag-new-version icon indicating copy to clipboard operation
action-detect-and-tag-new-version copied to clipboard

Use GITHUB_HEAD_REF if available

Open deeninetyone opened this issue 1 year ago • 5 comments

Fix #6 by using GITHUB_HEAD_REF if available

deeninetyone avatar Apr 20 '23 13:04 deeninetyone

Hi @deeninetyone, thanks for the PR! Can you describe what you're running into that requires the --openssl-legacy-provider flag?

dfreeman avatar Apr 21 '23 12:04 dfreeman

Hi @deeninetyone, thanks for the PR! Can you describe what you're running into that requires the --openssl-legacy-provider flag?

Hi there! This is the error I'm getting:

$ ncc build
ncc: Version 0.20.5
ncc: Compiling file index.js
Error: error:0308010C:digital envelope routines::unsupported
    at new Hash (node:internal/crypto/hash:71:[19](https://github.com/deeninetyone/action-detect-and-tag-new-version/actions/runs/4754881998/jobs/8448269803#step:4:20))
    at Object.createHash (node:crypto:133:10)
    at hashOf (evalmachine.<anonymous>:1:1478571)
    at module.exports (evalmachine.<anonymous>:1:1480625)
    at runCmd (evalmachine.<anonymous>:1:46940)
    at 819 (evalmachine.<anonymous>:1:43967)
    at __webpack_require__ (evalmachine.<anonymous>:1:[21](https://github.com/deeninetyone/action-detect-and-tag-new-version/actions/runs/4754881998/jobs/8448269803#step:4:22)6)
    at startup (evalmachine.<anonymous>:1:353)
    at module.exports.8 (evalmachine.<anonymous>:1:385)
    at evalmachine.<anonymous>:1:395 {
  opensslErrorStack: [ 'error:0[30](https://github.com/deeninetyone/action-detect-and-tag-new-version/actions/runs/4754881998/jobs/8448269803#step:4:31)00086:digital envelope routines::initialization error' ],
  library: 'digital envelope routines',
  reason: 'unsupported',
  code: 'ERR_OSSL_EVP_UNSUPPORTED'
}
error Command failed with exit code 1.
info Visit https://yarnpkg.com/en/docs/cli/run for documentation about this command.
error Command failed with exit code 1.
info Visit https://yarnpkg.com/en/docs/cli/run for documentation about this command.
Error: Process completed with exit code 1.

I've marked this WIP because the change assumes trying to get GITHUB_HEAD_REF will throw an exception if it's undefined, but seems that's not true from further testing

deeninetyone avatar Apr 21 '23 13:04 deeninetyone

Right, this works for me. There's a test I can't get to pass, and I'm not sure why

deeninetyone avatar Apr 21 '23 14:04 deeninetyone

Our project would benefit from this being merged as well! @dfreeman any chance of this happening?

bmiselis avatar Aug 03 '23 11:08 bmiselis

Any chance this will be merged? Also needing this.

DeanLa avatar May 20 '24 05:05 DeanLa