Skip to content

Conversation

dwsmart
Copy link
Contributor

@dwsmart dwsmart commented Jun 20, 2025

Makes progress on #4081

The SQL queries for the 2025 Page Weight Chapter

@dwsmart dwsmart self-assigned this Jul 30, 2025
@dwsmart dwsmart added the analysis Querying the dataset label Aug 18, 2025
Copy link
Member

@tunetheweb tunetheweb left a comment

Choose a reason for hiding this comment

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

Mostly LGTM. However with the move to JSON columns we can access fields directly (except when dashes are used, in which case need to use array-like accessors).

Also CrUX data is page-level. Just make sure that's what you want. And willing to live with some sites not having this.

Let me know when ready to merge.

@dwsmart dwsmart marked this pull request as ready for review September 1, 2025 10:13
@dwsmart
Copy link
Contributor Author

dwsmart commented Sep 1, 2025

Let me know when ready to merge.

I am ready from my side.

Copy link
Member

@tunetheweb tunetheweb left a comment

Choose a reason for hiding this comment

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

LGTM. Can I merge?

@dwsmart
Copy link
Contributor Author

dwsmart commented Sep 1, 2025

LGTM. Can I merge?

Happy my end!

@tunetheweb tunetheweb merged commit b817cb4 into HTTPArchive:main Sep 1, 2025
1 check passed
@tunetheweb
Copy link
Member

Cheers @dwsmart . Excellent work as always. Feel free to open follow up PRs as necessary as the chapter progresses.

@dwsmart dwsmart deleted the page-weight-sql-queries-2025 branch September 1, 2025 12:28
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
analysis Querying the dataset
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants