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

Remove transitive mapping tests? #13

Open
takikawa opened this issue Jun 25, 2024 · 2 comments
Open

Remove transitive mapping tests? #13

takikawa opened this issue Jun 25, 2024 · 2 comments

Comments

@takikawa
Copy link
Collaborator

In the source map hackfest, we discussed that at least browser consumers are unlikely to want to implement transitive mapping (mapping in which source maps map to original sources that themselves have mappings).

We currently do have tests for this case, but maybe we should remove them?

@nicolo-ribaudo
Copy link
Member

I think it's valuable to test tools that perform source map composition, and even though the current transitive mappings test is not optimal it maybe helps with that?

@dcrousso
Copy link
Member

dcrousso commented Oct 8, 2024

btw im pretty sure WebKit doesn't support this

it would likely require Web Inspector to actually parse the sourceContent in order to pull out the source map directive/URL instead of just displaying it

unless this is something super commonly used, id be in favor of removing it and not supporting it as that's a nontrivial amount of work to implement

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

3 participants