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

Debug IDs: Only allow Debug IDs in the canonical UUID format #144

Merged
merged 5 commits into from
Nov 15, 2024

Conversation

lforst
Copy link
Member

@lforst lforst commented Oct 29, 2024

Builds on #141


This proposal change may be a bit controversial.

I am suggesting to limit Debug IDs within Source Maps and generated files to be in a specific format - specifically the canonical UUID format (e.g. 85314830-023f-4cf1-a267-535f4e37bb17)

This is for the very simple reason of ensuring compatibility, consistency and simplicity in the ecosystem.

Drawbacks are:

  • Reduced flexibility
  • 4 more bytes per file

Would be great to have some feedback and discussions around this!

@nicolo-ribaudo
Copy link
Member

nicolo-ribaudo commented Nov 13, 2024

Could you rebase? :)

Copy link
Member

@nicolo-ribaudo nicolo-ribaudo left a comment

Choose a reason for hiding this comment

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

Given that we talked about this in the meeting and nobody raised concerns, and that it's an early stage proposal, I'm merging without further approvals.

@nicolo-ribaudo nicolo-ribaudo merged commit f768429 into tc39:main Nov 15, 2024
2 checks passed
github-actions bot added a commit that referenced this pull request Nov 15, 2024
SHA: f768429
Reason: push, by nicolo-ribaudo

Co-authored-by: github-actions[bot] <41898282+github-actions[bot]@users.noreply.github.com>
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

Successfully merging this pull request may close these issues.

2 participants