Skip to content

Commit

Permalink
remove references to dist directory from readme. these files should h…
Browse files Browse the repository at this point in the history
…opefully be able to be found via exports
  • Loading branch information
benmccann committed Jul 20, 2023
1 parent 1180131 commit d9423e0
Showing 1 changed file with 4 additions and 4 deletions.
8 changes: 4 additions & 4 deletions README.md
Original file line number Diff line number Diff line change
Expand Up @@ -85,7 +85,7 @@ Add the following to your Jest config:
```json
{
"transform": {
"^.+\\.svelte$": "svelte-jester/dist/transformer.cjs"
"^.+\\.svelte$": "svelte-jester"
},
"moduleFileExtensions": ["js", "svelte"]
}
Expand Down Expand Up @@ -219,7 +219,7 @@ path to the config file to the `preprocess` option thus:
```json
"transform": {
"^.+\\.svelte$": [
"svelte-jester/dist/transformer.cjs",
"svelte-jester",
{
"preprocess": true
}
Expand All @@ -240,7 +240,7 @@ path to the config file to the `preprocess` option thus:
```json
"transform": {
"^.+\\.svelte$": [
"svelte-jester/dist/transformer.cjs",
"svelte-jester",
{
"preprocess": "/some/path/to/svelte.config.js"
}
Expand Down Expand Up @@ -307,7 +307,7 @@ When `upward` is set it will stop at the first config file it finds above the fi
```json
"transform": {
"^.+\\.js$": "babel-jest",
"^.+\\.svelte$": ["svelte-jester/dist/transformer.cjs", {
"^.+\\.svelte$": ["svelte-jester", {
"preprocess": false,
"debug": false,
"compilerOptions": {},
Expand Down

3 comments on commit d9423e0

@sebastianrothe
Copy link
Collaborator

Choose a reason for hiding this comment

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

In the past, there were loads of problems with Jest, Node, Babel and Typescript all choosing the correct export of the library.

@benmccann
Copy link
Collaborator Author

Choose a reason for hiding this comment

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

Yeah, I found that Jest always prefers the CJS version. I reverted this commit and the related ones

@sebastianrothe
Copy link
Collaborator

Choose a reason for hiding this comment

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

It is a pity though :(

Please sign in to comment.