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): update dependency svgdom to v0.1.19 #106

Draft
wants to merge 1 commit into
base: main
Choose a base branch
from

Conversation

renovate[bot]
Copy link

@renovate renovate bot commented Oct 10, 2023

This PR contains the following updates:

Package Change Age Adoption Passing Confidence
svgdom 0.1.16 -> 0.1.19 age adoption passing confidence

Release Notes

svgdotjs/svgdom (svgdom)

v0.1.19

Compare Source

v0.1.18

Compare Source

v0.1.17

Compare Source


Configuration

📅 Schedule: Branch creation - At any time (no schedule defined), Automerge - At any time (no schedule defined).

🚦 Automerge: Disabled by config. Please merge this manually once you are satisfied.

Rebasing: Whenever PR becomes conflicted, or you tick the rebase/retry checkbox.

🔕 Ignore: Close this PR and you won't be reminded about this update again.


  • If you want to rebase/retry this PR, check this box

This PR was generated by Mend Renovate. View the repository job log.

@alexrudd2
Copy link
Owner

/Users/a.ruddick/Documents/github/saxi/dist/server/cli.js:42
const svgdom_1 = require("svgdom");
                 ^

Error [ERR_REQUIRE_ESM]: require() of ES Module /Users/a.ruddick/Documents/github/saxi/node_modules/svgdom/main-module.js from /Users/a.ruddick/Documents/github/saxi/dist/server/cli.js not supported.
Instead change the require of main-module.js in /Users/a.ruddick/Documents/github/saxi/dist/server/cli.js to a dynamic import() which is available in all CommonJS modules.
    at Object.<anonymous> (/Users/a.ruddick/Documents/github/saxi/dist/server/cli.js:42:18)
    at Object.<anonymous> (/Users/a.ruddick/Documents/github/saxi/cli.js:2:1) {
  code: 'ERR_REQUIRE_ESM'

@alexrudd2 alexrudd2 marked this pull request as draft October 10, 2023 16:44
@alexrudd2
Copy link
Owner

@jedahan Presumably you merged this to avoid svgdotjs/svgdom#114, but it breaks my install so I reverted until I can figure out what broke.

jedahan

This comment was marked as outdated.

@jedahan
Copy link
Collaborator

jedahan commented Oct 10, 2023

Got it - I've been meaning to review our tsconfig and esbuild settings which might help un-break this.

I wonder if we can reproduce this error using github actions / tests

@renovate renovate bot force-pushed the renovate/svgdom-0.x branch from 3911bdb to 70600e6 Compare October 12, 2023 23:11
@renovate renovate bot changed the title fix(deps): update dependency svgdom to v0.1.17 fix(deps): update dependency svgdom to v0.1.18 Oct 16, 2023
@renovate renovate bot force-pushed the renovate/svgdom-0.x branch from 70600e6 to ed77686 Compare October 16, 2023 10:06
@renovate renovate bot changed the title fix(deps): update dependency svgdom to v0.1.18 fix(deps): update dependency svgdom to v0.1.19 Oct 23, 2023
@renovate renovate bot force-pushed the renovate/svgdom-0.x branch from ed77686 to 7649b04 Compare October 23, 2023 19:31
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