Skip to content
This repository has been archived by the owner on Nov 18, 2021. It is now read-only.

Enable typescript-eslint/explicit-function-return-type #64

Open
wants to merge 1 commit into
base: master
Choose a base branch
from

Conversation

tetsuharuohzeki
Copy link
Contributor

We should annotate types of in/out of functions because programms are built with a bunch of functions.

It makes type safe world easy to annotate types of in/out of functions and we should express the type signature as documentation of their functions.

However, this rule is pretty tired for expressions because we use function expression as a callback. Let's disable its case.

@tetsuharuohzeki tetsuharuohzeki added A-config AREA: changes related to rules I-enhancement I-papercut IMPACT: Small but painful S-blocked-on-external C-upstream CONDITION: The issue needs to be resolved in the upstream side that this repository depends upon and removed I-enhancement C-upstream CONDITION: The issue needs to be resolved in the upstream side that this repository depends upon labels Mar 6, 2019
@tetsuharuohzeki
Copy link
Contributor Author

We need wait to resolve some problems in the upstream side:

  1. Don't warn about the case of const a = (args) => { ... };. I'd like to suppress this rule for this case,

@tetsuharuohzeki tetsuharuohzeki added this to the Support TypeScript milestone Mar 6, 2019
@github-actions
Copy link

github-actions bot commented Mar 6, 2019

☔ The latest upstream change (presumably these) made this pull request unmergeable. Please resolve the merge conflicts.

We should annotate types of in/out of functions
because programms are built with a bunch of functions.

It makes type safe world easy to annotate types of in/out of functions.
and we should express the type signature as documentation of their functions.

However, this rule is pretty tired for expressions because we use function expression
as a callback. Let's disable its case.
@github-actions
Copy link

☔ The latest upstream change (presumably these) made this pull request unmergeable. Please resolve the merge conflicts.

@tetsuharuohzeki tetsuharuohzeki added A-config/typescript and removed A-config AREA: changes related to rules labels Sep 26, 2019
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

1 participant