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(web): import helmet from node_modules #9927

Merged
merged 1 commit into from
Jan 29, 2024

Conversation

jtoar
Copy link
Contributor

@jtoar jtoar commented Jan 29, 2024

We can import this straight from node_modules instead of loading all the index which imports a bunch of other files and node_modules first.

@jtoar jtoar added the release:fix This PR is a fix label Jan 29, 2024
@jtoar jtoar added this to the v7.0.0 milestone Jan 29, 2024
@jtoar jtoar enabled auto-merge (squash) January 29, 2024 14:23
@jtoar jtoar disabled auto-merge January 29, 2024 14:52
@jtoar jtoar merged commit 472c86b into main Jan 29, 2024
38 checks passed
@jtoar jtoar deleted the ds-web/import-helmet-from-node-modules branch January 29, 2024 14:52
jtoar added a commit that referenced this pull request Jan 29, 2024
We can import this straight from node_modules instead of loading all the
index which imports a bunch of other files and node_modules first.
jtoar added a commit that referenced this pull request Jan 29, 2024
We can import this straight from node_modules instead of loading all the
index which imports a bunch of other files and node_modules first.
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
release:fix This PR is a fix
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants