Tony Brix

Results 617 comments of Tony Brix

@tststs 🙃

@tststs you can close this pr if you don't want it

> We haven't implemented this feature yet, because the use cases for unspy that I have seen have all been solvable with existing Jasmine functionality. Everything jasmine can do can...

My point is that this is obviously a wanted feature, even if it is just for semantics. My pull request uses the feature already in place to clear the spies...

This already had a PR #1289 @slackersoft doesn't seem to want people to restore a spy

You can use [jasmine-unspy](https://www.npmjs.com/package/jasmine-unspy) to add a method to remove the spy.

What do you get when you log the output of `require('marked')`?

I tried with a new cra app and `import { marked } from 'marked'` seems to work fine. Can you create a minimal repo that reproduces it? steps to reproduce:...

> `const marked = require('marked')` returns `/static/media/marked.af4dd239aa577a4f71e4.cjs`. This seems like an issue in your application that `require` is returning a path instead of importing. Most likely require is getting overridden...

`commonjs` import seems to work for me ([demo](https://runkit.com/uzitech/62d87f4db413f20008acddc6)). It looks like the issue is somewhere in create-react-app.