Skip to content

Commit

Permalink
fix(ci-output): properly cache dynamic info frames
Browse files Browse the repository at this point in the history
The `CIMultiStageOutput` class was caching dynamic blocks by their
label, then calculating wether or not to print new info based on the
throttle time and causing it to miss printing new dynamic stage data if the
throttle time is bigger than the time new data arrives.
  • Loading branch information
cristiand391 committed Dec 12, 2024
1 parent 9911c38 commit c7beaf4
Showing 1 changed file with 1 addition and 1 deletion.
2 changes: 1 addition & 1 deletion src/multi-stage-output.tsx
Original file line number Diff line number Diff line change
Expand Up @@ -294,7 +294,7 @@ class CIMultiStageOutput<T extends Record<string, unknown>> {

const formattedData = info.get ? info.get(this.data as T) : undefined
if (!formattedData) continue
const key = info.type === 'message' ? formattedData : info.label
const key = info.type === 'message' ? formattedData : `${info.label}: ${formattedData}`
const str = info.type === 'message' ? formattedData : `${info.label}: ${formattedData}`

const lastUpdateTime = this.lastUpdateByInfo.get(key)
Expand Down

0 comments on commit c7beaf4

Please sign in to comment.