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

Is the custom inspect() still needed? #74

Open
matt-usurp opened this issue Nov 11, 2023 · 0 comments
Open

Is the custom inspect() still needed? #74

matt-usurp opened this issue Nov 11, 2023 · 0 comments

Comments

@matt-usurp
Copy link

matt-usurp commented Nov 11, 2023

I have an issue raised upstream with vitest which had me do some digging here, I am wondering if the custom inspect() function is still required? All the build configuration files I can see here suggest that this package is targeting esm and esnext which will have symbol support. That and the recent https://github.com/chaijs/loupe/releases/tag/v3.0.0 release seem to back this up.

Specifically, I am asking if the named inspect() function is still required when we have the two symbol https://github.com/chaijs/loupe/blob/main/src/index.ts#L99C7-L99C18 variants available to us.

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

No branches or pull requests

1 participant