Skip to content
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

fix(deps): Align and integrity-check cdn resources in perf trials #918

Open
wants to merge 2 commits into
base: main
Choose a base branch
from

Conversation

legobeat
Copy link
Contributor

  • chore: add integrity and crossorigin attributes to js and stylesheets loaded from external CDN
  • fix(deps): bump d3 from v3 to v5 (latest supported by c3)

c3 is unmaintained and if updating d3 for the perf trials further is actually desired, it would probably have to be replaced with something else, like billboard.js

@github-actions github-actions bot added the pkg:lavamoat-perf Changes in package lavamoat-perf label Jan 19, 2024
@legobeat legobeat changed the title fix(perf): Align and integrity-check cdn resources fix(deps): Align and integrity-check cdn resources Jan 19, 2024
@legobeat legobeat marked this pull request as ready for review January 19, 2024 09:58
@legobeat legobeat requested a review from a team as a code owner January 19, 2024 09:58
@legobeat legobeat changed the title fix(deps): Align and integrity-check cdn resources fix(deps): Align and integrity-check cdn resources in perf trials Jan 19, 2024
@legobeat legobeat force-pushed the deps-perf-d3 branch 7 times, most recently from 3b3bd75 to 14c5c86 Compare January 23, 2024 08:24
@boneskull
Copy link
Contributor

@kumavis pls help

@legobeat legobeat force-pushed the deps-perf-d3 branch 5 times, most recently from 1485edc to 53d11c0 Compare February 5, 2024 10:00
@boneskull
Copy link
Contributor

I'd approve this, but I'm still waiting for that CORS talk from @naugtur

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
pkg:lavamoat-perf Changes in package lavamoat-perf
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

2 participants