Skip to content

Conversation

@kkartunov
Copy link
Collaborator

Related JIRA Ticket:

https://topcoder.atlassian.net/browse/

What's in this PR?

@kkartunov kkartunov requested a review from jmgasper as a code owner January 29, 2026 17:23
}
}, [fetchError])

const uniqueRuns = uniqBy(orderBy(runs, 'completedAt', 'desc'), 'workflow.id')

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

[⚠️ performance]
The reverse() method is used after orderBy, which might be unnecessary if the intention is to sort in descending order. Consider removing reverse() and using orderBy(runs, 'completedAt', 'asc') if ascending order is desired.

Comment out the running_yarn_sb_build step in CircleCI config
- run: *build_configuration_fetch
- run: *running_yarn_build
- run: *running_yarn_sb_build
# - run: *running_yarn_sb_build

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

[⚠️ maintainability]
The running_yarn_sb_build step has been commented out. Ensure that this is intentional and that the Storybook build is no longer needed in the build process. If it is still required, consider implementing a conditional execution based on environment or branch to avoid unnecessary builds.

@kkartunov kkartunov merged commit c953b0a into master Jan 29, 2026
3 of 5 checks passed
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.

3 participants