-
-
Notifications
You must be signed in to change notification settings - Fork 23
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Upgrade to new datamodel #209
Conversation
Co-authored-by: Max Ostapenko <[email protected]>
Think this is good to merge. It converts all reports and also simplifies the lens (particularly for blink_fetaures) considerably. We could further simplify the CrUX ones to always join to The only report that doesn't work with lenses is @max-ostapenko since you already spotted one typo do you wanna have a look over before I merge? Hopefully the queries will be easier to migrate to your better solution and we can stop using that hack of a shell script soon. |
Thanks for the effort here. |
WIP as I rerun the reports