Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Remove unused type #328

Merged
merged 1 commit into from
Nov 29, 2021
Merged

Remove unused type #328

merged 1 commit into from
Nov 29, 2021

Conversation

antoniopresto
Copy link
Contributor

@antoniopresto antoniopresto commented Nov 29, 2021

Fix importing these types while using noUnusedLocals config, since skipLibCheck can't prevent this.

fix importing these type while using ts noUnusedLocals config (skipLibCheck can't prevent this).
@codesandbox-ci
Copy link

This pull request is automatically built and testable in CodeSandbox.

To see build info of the built libraries, click here or the icon next to each commit SHA.

Latest deployment of this branch, based on commit e50ba8a:

Sandbox Source
Vanilla Configuration
Vanilla Typescript Configuration

@timdorr
Copy link
Member

timdorr commented Nov 29, 2021

Thanks!

@timdorr timdorr merged commit 84128f0 into reduxjs:master Nov 29, 2021
@antoniopresto antoniopresto deleted the patch-1 branch November 29, 2021 22:33
@uncaught
Copy link

uncaught commented May 9, 2022

@timdorr could you please push for a new release? Somehow my question (#334) about it got deleted without any response.

@lriho
Copy link

lriho commented Sep 1, 2022

Just pinging here as I'd also like to see a new release. We had to disable noUnusedLocals because of that.

@aniravi24
Copy link

aniravi24 commented Oct 14, 2022

Not to pile on here, but would love to see a patch release for this as well! Running into the same thing.

@antoniopresto
Copy link
Contributor Author

Released :D ea2c5dd

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.

5 participants