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

breaking(middleware/devtools): remove deprecations and warnings #892

Merged
merged 141 commits into from Apr 8, 2022

Conversation

devanshj
Copy link
Contributor

@devanshj devanshj commented Apr 5, 2022

remove deprecated features for v4.

@codesandbox-ci
Copy link

codesandbox-ci bot commented Apr 5, 2022

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 f991f00:

Sandbox Source
React Configuration
React Typescript Configuration
React Browserify Configuration
React Snowpack Configuration
React Parcel Configuration
Next.js Configuration

@dai-shi dai-shi changed the title devtools: remove deprecations and warnings breaking(devtools): remove deprecations and warnings Apr 5, 2022
@dai-shi dai-shi added this to the v4.0.0 milestone Apr 7, 2022
@dai-shi dai-shi changed the title breaking(devtools): remove deprecations and warnings breaking(middleware/devtools): remove deprecations and warnings Apr 7, 2022
@dai-shi
Copy link
Member

dai-shi commented Apr 7, 2022

@devanshj Please change the base branch to main.

@devanshj devanshj changed the base branch from v4 to main April 7, 2022 21:31
@devanshj
Copy link
Contributor Author

devanshj commented Apr 7, 2022

Not sure why the build is failing, perhaps I've removed some changes that are needed.

@dai-shi
Copy link
Member

dai-shi commented Apr 7, 2022

Not sure why it passed when we add __DEV__. Maybe, it was incorrect CI setup.

}),
{ count: 0 }
describe('with redux middleware', () => {
let api: any
Copy link
Member

Choose a reason for hiding this comment

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

Do you have a better idea than any?

Copy link
Contributor Author

@devanshj devanshj Apr 8, 2022

Choose a reason for hiding this comment

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

Wouldn't let api = create(...) just work? Let me try

Edit: hmm weird doesn't work, not sure how this is different tho

Copy link
Member

Choose a reason for hiding this comment

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

It is because of beforeEach.

Copy link
Member

@dai-shi dai-shi left a comment

Choose a reason for hiding this comment

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

LGTM. Let's merge.

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.

None yet

2 participants