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

chore: npm update #812

Merged
merged 1 commit into from
Feb 15, 2024
Merged

chore: npm update #812

merged 1 commit into from
Feb 15, 2024

Conversation

usefulthink
Copy link
Contributor

@usefulthink usefulthink commented Sep 27, 2023

Manually update all dependencies.

@usefulthink usefulthink force-pushed the chore/npm-update branch 7 times, most recently from 3f8bfb0 to 9fa3d4e Compare September 28, 2023 10:38
@usefulthink usefulthink requested a review from wangela November 6, 2023 08:11
@wangela wangela merged commit 3d221ea into googlemaps:main Feb 15, 2024
14 checks passed
github-actions bot pushed a commit that referenced this pull request Feb 15, 2024
usefulthink added a commit to usefulthink/js-api-loader that referenced this pull request Feb 16, 2024
in googlemaps#812 the package type was accidentally changed to "module" which leads to all sorts of problems. This is reverted and config-files that are already in esm-format are renamed to .mjs to continue working.
kwnevarez pushed a commit that referenced this pull request Feb 16, 2024
in #812 the package type was accidentally changed to "module" which leads to all sorts of problems. This is reverted and config-files that are already in esm-format are renamed to .mjs to continue working.
github-actions bot pushed a commit that referenced this pull request Feb 16, 2024
in #812 the package type was accidentally changed to "module" which leads to all sorts of problems. This is reverted and config-files that are already in esm-format are renamed to .mjs to continue working. fce8110
@@ -33,38 +34,41 @@
"test:e2e": "jest e2e/*"
},
"dependencies": {
"@rollup/plugin-terser": "^0.4.3",

Choose a reason for hiding this comment

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

shouldn't this be a dev dependency?

Copy link
Contributor Author

Choose a reason for hiding this comment

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

It should, thanks for pointing this out.

Copy link
Member

Choose a reason for hiding this comment

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

fixed in #830

@usefulthink usefulthink deleted the chore/npm-update branch February 16, 2024 22:25
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.

3 participants