-
Notifications
You must be signed in to change notification settings - Fork 17
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
Hardening: version field #26
Labels
Comments
This was referenced Apr 6, 2023
mitsuhiko
added a commit
to tc39/source-map-spec
that referenced
this issue
Jul 5, 2023
mitsuhiko
added a commit
to tc39/source-map-spec
that referenced
this issue
Jul 6, 2023
We agreed on requiring the value to be |
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Today the specification says this about version:
It does not say that this is the source map spec version, but it implies this by showing the integer
3
everywhere for the source map version (v3). It could also be a running counter. It also is documented to come first in the file which in practice is not enforced and hard to actually enforce given that many languages only have unsorted hash maps.Would it make sense to drop the ordering requirement and to mention that this is explicitly the version of the source map, and what meaning it has?
My recommendation would be:
The text was updated successfully, but these errors were encountered: