Skip to content

Conversation

@brandenrodgers
Copy link
Contributor

Description and Context

porting hs cms lighthouse-score to TS.

This relies on an LDL PR: HubSpot/hubspot-local-dev-lib#268

Screenshots

TODO

Who to Notify

@brandenrodgers brandenrodgers changed the title port cms lighthousr-score to TS port cms lighthouse-score to TS May 9, 2025
@brandenrodgers brandenrodgers changed the title port cms lighthouse-score to TS port cms lighthouse-score to TS (waiting for LDL release) May 9, 2025
camden11
camden11 previously approved these changes May 9, 2025

const getTableData = (target, scoreResult) => {
const scores = scoreResult.scores ? scoreResult.scores[0] : {};
const getTableData = (
Copy link
Contributor

Choose a reason for hiding this comment

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

nit: use function here

@brandenrodgers brandenrodgers merged commit b1c8f3c into main May 9, 2025
1 check passed
@brandenrodgers brandenrodgers deleted the br/ts-lighthouse-score branch May 9, 2025 20:29
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.

4 participants