Skip to content
This repository has been archived by the owner on Jun 24, 2024. It is now read-only.

Updated the spec to clearly define usage of the sourcemap comment formats #23

Merged
merged 2 commits into from
Jan 18, 2024

Conversation

bomsy
Copy link
Collaborator

@bomsy bomsy commented Nov 13, 2023

For #54. Updated the supported comments based of the decisions at the last meeting.

@nicolo-ribaudo
Copy link
Member

Thanks!

Regarding the failing IPR check: ideally you should be onboarded as a Mozilla delegate so that it's explicit that your contribution are on behalf of your company (cc @dminor could it be done? 🙂), but if it will be a blocker for merging this change we can temporary add an exception for your username.

source-map.bs Outdated Show resolved Hide resolved
@bomsy
Copy link
Collaborator Author

bomsy commented Nov 13, 2023

Thanks!

Regarding the failing IPR check: ideally you should be onboarded as a Mozilla delegate so that it's explicit that your contribution are on behalf of your company (cc @dminor could it be done? 🙂), but if it will be a blocker for merging this change we can temporary add an exception for your username.

What do i need to do to onboard?

@nicolo-ribaudo
Copy link
Member

@bomsy is now a TC39 delegate, so IPR is good :)

@nicolo-ribaudo nicolo-ribaudo force-pushed the supported-comment-formats branch from 7d9a686 to e2fe1b3 Compare January 10, 2024 17:28
@jkup jkup merged commit fabdabd into tc39:main Jan 18, 2024
2 checks passed
github-actions bot added a commit that referenced this pull request Jan 18, 2024
SHA: fabdabd
Reason: push, by jkup

Co-authored-by: github-actions[bot] <41898282+github-actions[bot]@users.noreply.github.com>
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants