Skip to content

chore: upgrade @google-cloud/storage to 7.8.0 (stream-shift issue)#2

Open
MPV wants to merge 2 commits intopass-culture-github-actions:mainfrom
pagero:upgrade-google-cloud-storage-to-fix-issue-in-stream-shift
Open

chore: upgrade @google-cloud/storage to 7.8.0 (stream-shift issue)#2
MPV wants to merge 2 commits intopass-culture-github-actions:mainfrom
pagero:upgrade-google-cloud-storage-to-fix-issue-in-stream-shift

Conversation

@MPV
Copy link

@MPV MPV commented Jul 4, 2024

Here is the desired fix:

To fix the issue giving:

  /home/runner/_work/_actions/pass-culture-github-actions/gcs-cache/4857aab4515c00e04051f4da50c145f2a1d11019/dist/webpack:/gcs-cache/node_modules/stream-shift/index.js:16
      return state.buffer[0].length
  ^
  TypeError: Cannot read properties of null (reading 'length')
      at getStateLength (/home/runner/_work/_actions/pass-culture-github-actions/gcs-cache/4857aab4515c00e04051f4da50c145f2a1d11019/dist/webpack:/gcs-cache/node_modules/stream-shift/index.js:16:1)
      at shift (/home/runner/_work/_actions/pass-culture-github-actions/gcs-cache/4857aab4515c00e04051f4da50c145f2a1d11019/dist/webpack:/gcs-cache/node_modules/stream-shift/index.js:6:1)
      at Duplexify._forward (/home/runner/_work/_actions/pass-culture-github-actions/gcs-cache/4857aab4515c00e04051f4da50c145f2a1d11019/dist/webpack:/gcs-cache/node_modules/duplexify/index.js:170:1)
      at PassThrough.onreadable (/home/runner/_work/_actions/pass-culture-github-actions/gcs-cache/4857aab4515c00e04051f4da50c145f2a1d11019/dist/webpack:/gcs-cache/node_modules/duplexify/index.js:136:1)
      at PassThrough.emit (node:events:519:28)
      at emitReadable_ (node:internal/streams/readable:832:12)
      at processTicksAndRejections (node:internal/process/task_queues:81:21)

@enricojonas
Copy link

Can someone merge this or do we need yet another fork ❓

@MPV
Copy link
Author

MPV commented Jan 30, 2025

Can someone merge this or do we need yet another fork ❓

Already moved on myself, to:

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants