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: more vite.config cleanups #1589

Merged
merged 5 commits into from
Sep 3, 2024

Conversation

ValeryG
Copy link
Collaborator

@ValeryG ValeryG commented Aug 29, 2024

Summary

removing more entities covered in shared

@ValeryG ValeryG requested review from a team as code owners August 29, 2024 15:12
ValeryG and others added 4 commits August 29, 2024 09:36
Co-authored-by: Adam DeHaven <2229946+adamdehaven@users.noreply.github.com>
Co-authored-by: Adam DeHaven <2229946+adamdehaven@users.noreply.github.com>
@ValeryG ValeryG requested a review from adamdehaven August 29, 2024 15:51
@adamdehaven adamdehaven merged commit 299cd22 into feat/entities-packaging Sep 3, 2024
2 checks passed
@adamdehaven adamdehaven deleted the feat/entities-packaging-3 branch September 3, 2024 15:07
adamdehaven added a commit that referenced this pull request Sep 4, 2024
* feat(entities-certificates): externalize deps

* refactor: use regular expression for package match

* refactor: use regular expression for package match

* fix(deps): externalize for all packages

* fix(rollup): externalize

* fix: package.json

* fix(deps): use workspace version

* fix: lockfile

* fix: external dep pattern match

* fix: externals

* chore: exclude styles

* chore: remove duplicate entries covered by root config

* fix: do not externalize styles

* fix: more vite.config cleanups (#1589)

* fix: more vite.config cleanups

* Update packages/entities/entities-routes/vite.config.ts

Co-authored-by: Adam DeHaven <2229946+adamdehaven@users.noreply.github.com>

* Update packages/analytics/analytics-geo-map/vite.config.ts

Co-authored-by: Adam DeHaven <2229946+adamdehaven@users.noreply.github.com>

* Update vite.config.ts

* Update vite.config.ts

---------

Co-authored-by: Adam DeHaven <2229946+adamdehaven@users.noreply.github.com>

---------

Co-authored-by: ValeryG <vvalggor@gmail.com>
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.

2 participants