bump ecmarkdown and format valueless attributes

tc39 · Apr 7, 2023 · 91cc4e5 · 91cc4e5
1 parent 2054ff8
commit 91cc4e5
Show file tree

Hide file tree

Showing 4 changed files with 13 additions and 11 deletions.
diff --git a/package-lock.json b/package-lock.json
diff --git a/package.json b/package.json
@@ -49,7 +49,7 @@
  "command-line-args": "^5.2.0",
  "command-line-usage": "^6.1.1",
  "dedent-js": "^1.0.1",
- "ecmarkdown": "^8.0.0",
+ "ecmarkdown": "^8.1.0",
  "eslint-formatter-codeframe": "^7.32.1",
  "fast-glob": "^3.2.7",
  "grammarkdown": "^3.2.0",

diff --git a/src/formatter/ecmarkdown.ts b/src/formatter/ecmarkdown.ts
@@ -43,7 +43,9 @@ async function printStep(
  output.firstLineIsPartial = false;
  output.appendText(item.name === 'ordered-list-item' ? '1. ' : '* ');
  if (item.attrs.length > 0) {
- const joined = item.attrs.map(({ key, value }) => `${key}=${JSON.stringify(value)}`).join(', ');
+ const joined = item.attrs
+ .map(({ key, value }) => (value === '' ? key : `${key}=${JSON.stringify(value)}`))
+ .join(', ');
  output.appendText(`[${joined}] `);
  }
  const contents = await printFragments(source, item.contents, indent + 1);

diff --git a/test/formatter.js b/test/formatter.js
@@ -473,7 +473,7 @@ describe('algorithm formatting', () => {
  1. Step.
  1. Another step.
  1. A substep.
- 1. [ x = "a",y="b" ] A step with attributes.
+ 1. [ x = "a",y="b" ,c="",d ] A step with attributes.
  1. A step with _vars_, numbers like **10<sup>x</sup>**, records like { [[A]]: 0 }, and fields like _o_.[[field]].
  </emu-alg>
  `,
@@ -482,7 +482,7 @@ describe('algorithm formatting', () => {
  1. Step.
  1. Another step.
  1. A substep.
- 1. [x="a", y="b"] A step with attributes.
+ 1. [x="a", y="b", c, d] A step with attributes.
  1. A step with _vars_, numbers like **10<sup>x</sup>**, records like { [[A]]: 0 }, and fields like _o_.[[field]].
  </emu-alg>
  `