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

refactor(fromFetch): remove redundant if wrap of init #5933

Merged
merged 1 commit into from Jan 3, 2021
Merged

refactor(fromFetch): remove redundant if wrap of init #5933

merged 1 commit into from Jan 3, 2021

Conversation

iamandrewluca
Copy link
Contributor

@iamandrewluca iamandrewluca commented Dec 14, 2020

Description:

init object is created using rest operator
this means that init always will be an object which is a truthy value
this commit removes redundant init check

Related: https://github.com/ReactiveX/rxjs/pull/5306/files#r542228847

`init` object is created using rest operator
this means that `init` always will be an object which is a truthy value
this commit removes redundant `init` check
@iamandrewluca
Copy link
Contributor Author

package-lock.json file was modified automatically when I executed npm install

@kwonoj
Copy link
Member

kwonoj commented Dec 14, 2020

this means that init always will be an object which is a truthy value

is it true? RequestInit's property is all optional so this can happen:

ugh, nvm I was incorrect.

@benlesh benlesh merged commit a85627f into ReactiveX:master Jan 3, 2021
@benlesh
Copy link
Member

benlesh commented Jan 3, 2021

thanks, @iamandrewluca

@iamandrewluca iamandrewluca deleted the remove-redundant-if branch January 3, 2021 21:25
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

4 participants