Skip to content

Improve UnableToEmitSwitch warning message #11680

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

Closed
wants to merge 1 commit into from

Conversation

japgolly
Copy link
Contributor

Previously it would read:

Could not emit switch for @switch annotated match since there are not enough cases

Now it reads:

Could not emit switch for @switch annotated match since a minimum of 4 cases (including the default) are required

Previously it would read:
  Could not emit switch for @switch annotated match since there are not enough cases

Now it reads:
  Could not emit switch for @switch annotated match since a minimum of 4 cases (including the default) are required
@smarter
Copy link
Member

smarter commented Mar 12, 2021

The change by itself looks fine but the open PR #9852 should completely get rid of this warning which we agreed was a better solution in #9776

@smarter
Copy link
Member

smarter commented Mar 12, 2021

Closed in favor of #9852

@smarter smarter closed this Mar 12, 2021
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