Skip to content

Remove react-draggable and move to TS6#868

Merged
loichuder merged 2 commits into
mainfrom
remove-draggable
Jun 9, 2026
Merged

Remove react-draggable and move to TS6#868
loichuder merged 2 commits into
mainfrom
remove-draggable

Conversation

@loichuder

Copy link
Copy Markdown
Member

No description provided.

Comment thread tsconfig.json
"noImplicitOverride": true,
"noImplicitReturns": true,
"noFallthroughCasesInSwitch": true,
"noUncheckedSideEffectImports": true

@loichuder loichuder Jun 9, 2026

Copy link
Copy Markdown
Member Author

Choose a reason for hiding this comment

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

Removed fields that are the same as the default value: https://devblogs.microsoft.com/typescript/announcing-typescript-6-0/

@loichuder loichuder changed the title Remove react-draggable Remove react-draggable and move to TS6 Jun 9, 2026
@loichuder loichuder requested a review from axelboc June 9, 2026 13:20
Comment thread tsconfig.json Outdated
"compilerOptions": {
"module": "esnext",
"moduleResolution": "bundler",
"target": "esnext",

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

target defaults to the current year, so I think you can remove it as well.

Comment thread tsconfig.json Outdated
"module": "esnext",
"moduleResolution": "bundler",
"target": "esnext",
"lib": ["dom", "dom.iterable", "esnext"],

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

Suggested change
"lib": ["dom", "dom.iterable", "esnext"],
"lib": ["dom", "esnext"],

https://devblogs.microsoft.com/typescript/announcing-typescript-6-0/

Copy link
Copy Markdown
Member Author

Choose a reason for hiding this comment

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

Missed this one, thanks!

@loichuder loichuder merged commit a6d76a4 into main Jun 9, 2026
2 checks passed
@loichuder loichuder deleted the remove-draggable branch June 9, 2026 14:38
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.

2 participants