Skip to content

🏗 Partial updates for core devDependencies#39765

Merged
danielrozenberg merged 4 commits intoampproject:mainfrom
danielrozenberg:core-devdep-partial-c-d
Jan 26, 2024
Merged

🏗 Partial updates for core devDependencies#39765
danielrozenberg merged 4 commits intoampproject:mainfrom
danielrozenberg:core-devdep-partial-c-d

Conversation

@danielrozenberg
Copy link
Copy Markdown
Member

@danielrozenberg danielrozenberg commented Jan 23, 2024

This PR updates code devDependencies that require non-trivial changes to the codebase:

  • dedent - explicitly requires that we import its default
  • commander - requires that we import Program instead of the default export
  • typescript - requires a minor type definition change in src/preact/component/types.d.ts

This PR also removes the why-is-node-running package and the debugging code that used it, which should fix the broken amp --help command

@danielrozenberg danielrozenberg force-pushed the core-devdep-partial-c-d branch 3 times, most recently from 129b80f to e1f9d8b Compare January 24, 2024 18:43
@danielrozenberg danielrozenberg marked this pull request as ready for review January 24, 2024 18:54
@danielrozenberg danielrozenberg force-pushed the core-devdep-partial-c-d branch from e1f9d8b to 1578197 Compare January 26, 2024 17:00
@danielrozenberg danielrozenberg enabled auto-merge (squash) January 26, 2024 17:00
@danielrozenberg danielrozenberg merged commit 74f443b into ampproject:main Jan 26, 2024
@danielrozenberg danielrozenberg deleted the core-devdep-partial-c-d branch January 26, 2024 17:22
eszponder pushed a commit to krzysztofequativ/amphtml that referenced this pull request Apr 22, 2024
* Remove CircleCI debugging code from unit.js

* Upgrade `commander` to v11.1.0

* Upgrade `dedent` to v1.5.1

* Upgrade `typescript` to v5.3.3
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants