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

[BUGFIX stable] TS lint-level error in mixins/-proxy.ts #20429

Merged
merged 1 commit into from
Apr 3, 2023

Conversation

chriskrycho
Copy link
Contributor

@chriskrycho chriskrycho commented Apr 3, 2023

Explicitly do a consistent return from a number of spots which TS 5.1 nightlies are catching, and which previous versions did not. In these cases, we explicitly return undefined, which is equivalent to the previous behavior.

Fixes a lint-style error with TS nightly.

@chriskrycho chriskrycho added internal TypeScript Work on Ember’s types labels Apr 3, 2023
@dfreeman
Copy link
Contributor

dfreeman commented Apr 3, 2023

Looks like there are two more instances flagged in @ember/debug and @ember/object

Explicitly do a consistent return from a number of spots which TS 5.1
nightlies are catching, and which previous versions did not. In these
cases, we explicitly return `undefined`, which is equivalent to the
previous behavior.
@chriskrycho chriskrycho added Bug and removed internal labels Apr 3, 2023
@chriskrycho chriskrycho changed the title Fix TS lint-level error in mixins/-proxy.ts [BUGFIX BETA] TS lint-level error in mixins/-proxy.ts Apr 3, 2023
@chriskrycho chriskrycho changed the title [BUGFIX BETA] TS lint-level error in mixins/-proxy.ts [BUGFIX stable] TS lint-level error in mixins/-proxy.ts Apr 3, 2023
@chriskrycho chriskrycho merged commit 16b1b97 into master Apr 3, 2023
@chriskrycho chriskrycho deleted the consistent-return branch April 3, 2023 14:49
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Bug TypeScript Work on Ember’s types
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants