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

Editorial: make changes requested by ECMA ExeCom #140

Closed
wants to merge 2 commits into from

Conversation

takikawa
Copy link
Collaborator

This PR contains editorial changes based on recent feedback from ECMA for the current draft. See the commit message for details.

  * Adjust the title
  * Moved "about this specification" to the introduction section
  * Add new scope language
  * Re-ordered sections: Scope, Conformance, References, Term and Definitions
  * Manually generate the references section
    - It doesn't seem possible to generate the references as a normal
      numbered section without this manual approach, which is a hack.
source-map.bs Outdated Show resolved Hide resolved
Comment on lines +223 to +233
<!--
NOTE: This references section is manually generated because bikeshed assumes
that the references section should be un-numbered and we want to include
it as a numbered section. To generate it, use the following and then
copy the inner HTML contents:

<div data-fill-with="references"></div>

and replace [ with \[ to escape macros. Also add data-no-self-link="" to
each <dt> entry.
-->
Copy link
Member

Choose a reason for hiding this comment

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

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.

Once merged, this will need to be ported to the 2024 branch.

@nicolo-ribaudo
Copy link
Member

I tried "close and reopen" because this PR was stuck while merging, and now I cannot reopen. @takikawa could you open a new PR from the same branch? 😅

@nicolo-ribaudo
Copy link
Member

Ok nevermind, it has been successfully merged even if GitHub says no

szuend pushed a commit to szuend/source-map that referenced this pull request Nov 4, 2024
Editorial: make changes requested by ECMA ExeCom
github-actions bot added a commit to szuend/source-map that referenced this pull request Nov 4, 2024
SHA: 8be0fe1
Reason: push, by szuend

Co-authored-by: github-actions[bot] <41898282+github-actions[bot]@users.noreply.github.com>
github-actions bot added a commit to szuend/source-map that referenced this pull request Nov 6, 2024
SHA: 8be0fe1
Reason: push, by szuend

Co-authored-by: github-actions[bot] <41898282+github-actions[bot]@users.noreply.github.com>
nicolo-ribaudo pushed a commit that referenced this pull request Nov 11, 2024
Editorial: make changes requested by ECMA ExeCom
github-actions bot added a commit to nicolo-ribaudo/source-map that referenced this pull request Nov 11, 2024
SHA: 8be0fe1
Reason: push, by nicolo-ribaudo

Co-authored-by: github-actions[bot] <41898282+github-actions[bot]@users.noreply.github.com>
github-actions bot added a commit that referenced this pull request Nov 11, 2024
SHA: 6dc57d3
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.

4 participants