8000 build: drop tsconfig options that typescript 6 makes default by morellodev · Pull Request #353 · awesome-reveal/awesome-reveal · GitHub
[go: up one dir, main page]

Skip to content

build: drop tsconfig options that typescript 6 makes default - #353

Merged
morellodev merged 1 commit into
nextfrom
build/ts6-tsconfig-defaults
Oct 25, 2026
Merged

morellodev merged 1 commit into
nextfrom
build/ts6-tsconfig-defaults

Conversation

@morellodev
Copy link
Copy Markdown
Collaborator

Summary

This PR removes two options from tsconfig.base.json that have no effect with TypeScript 6. The TypeScript 6.0 release notes describe both changes:

  • "DOM.Iterable" in lib: lib.dom.d.ts now contains the iterable declarations, and dom.iterable is an empty file.
  • "strict": true: strict is now true by default.

examples/next/tsconfig.json keeps the same options, because Next.js writes that file.

Checklist

  • The PR targets the next branch.
  • The PR title follows Conventional Commits. A breaking change has ! and a BREAKING CHANGE: footer.
  • Tests cover the change. These checks pass:
    • tsc --noEmit for the src of each package, tsc -p packages/<package>/test for core, react, react-awesome-reveal and astro, vue-tsc -p test for vue, tsc -p packages/test-utils and tsc -p .
    • astro check in examples/astro, vue-tsc in examples/vue and tsc in examples/react
    • A probe file in core still fails with TS2322 and TS7006, so strict mode stays on.
    • pnpm build from a clean state logs no TypeScript diagnostics, and each .d.ts file in dist is byte-identical to the output before this PR.
  • A change to an option, a default or an export also updates the READMEs, the consumer skills and the examples. Does not apply: no option, default or export of the packages changes.

@morellodev
morellodev merged commit 2b8288d into next Oct 25, 2026
19 checks passed
@morellodev
morellodev deleted the build/ts6-tsconfig-defaults branch October 25, 2026 07:51
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.

1 participant

0