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.

Sign up for GitHub

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

Jump to bottom

[core] Remove redundant production check #17929

Merged
merged 1 commit into from
Oct 18, 2019
Merged

[core] Remove redundant production check #17929

merged 1 commit into from
Oct 18, 2019

Conversation

ellisio
Copy link
Contributor

@ellisio ellisio commented Oct 18, 2019

This extra if condition is redundant.

@ellisio
remove redundant production check
38296c1 
This extra if condition is redundant.
@mui-pr-bot
Copy link

No bundle size changes comparing 0a6d9a5...38296c1

Generated by 🚫 dangerJS against 38296c1

@oliviertassinari oliviertassinari added core Infrastructure work going on behind the scenes PR: ready to ship labels Oct 18, 2019
@oliviertassinari oliviertassinari changed the title [core] remove redundant production check [core] Remove redundant production check Oct 18, 2019
@joshwooding joshwooding merged commit c99dd32 into mui:master Oct 18, 2019
@joshwooding
Copy link
Member

@ellisio Thanks 👍

@ellisio ellisio deleted the patch-1 branch October 22, 2019 16:38
@snyk-bot snyk-bot mentioned this pull request Jan 7, 2020
Open
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@oliviertassinari oliviertassinari oliviertassinari approved these changes

@joshwooding joshwooding joshwooding approved these changes

Assignees
No one assigned
Labels
core Infrastructure work going on behind the scenes
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
4 participants
@ellisio @mui-pr-bot @joshwooding @oliviertassinari