Skip to content

Commit 187d4d3

Browse files
chore: drop now-unused @ts-expect-error directives in serializeData
Problem: tsc --build fails on upstream/main with three TS2578 "Unused '@ts-expect-error' directive" errors at lib/index.js:233/235/237. markdown-table's types now accept null for the align-delimiters, padding, and string-length options, so the suppressions are obsolete and the type checker now flags them. Goal: restore tsc --build to a clean pass so the package's own test suite runs end-to-end without an unrelated build failure. Changes: - lib/index.js: remove three @ts-expect-error comments around the alignDelimiters / padding / stringLength fields passed to markdownTable in serializeData. Notes: drift from a markdown-table type update; verified npm test now passes (build + format + 100% coverage + 32 of 32 tests dev and prod).
1 parent abf8c36 commit 187d4d3

1 file changed

Lines changed: 0 additions & 3 deletions

File tree

lib/index.js

Lines changed: 0 additions & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -230,11 +230,8 @@ export function gfmTableToMarkdown(options) {
230230
function serializeData(matrix, align) {
231231
return markdownTable(matrix, {
232232
align,
233-
// @ts-expect-error: `markdown-table` types should support `null`.
234233
alignDelimiters,
235-
// @ts-expect-error: `markdown-table` types should support `null`.
236234
padding,
237-
// @ts-expect-error: `markdown-table` types should support `null`.
238235
stringLength
239236
})
240237
}

0 commit comments

Comments
 (0)