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

ik wil graag dat dit kan #4

Open
YouriLieverdink opened this issue Jul 28, 2023 · 6 comments
Open

ik wil graag dat dit kan #4

YouriLieverdink opened this issue Jul 28, 2023 · 6 comments
Assignees

Comments

@YouriLieverdink
Copy link
Contributor

export function getInitialPayout(
    woz: number,
    debt: number,
    age: number,
): number {
}
@YouriLieverdink
Copy link
Contributor Author

@QGIsK This rule has something to do with it:

"function-paren-newline": ["warn", "multiline-arguments"],

@Goosterhof
Copy link
Contributor

Ik mis wat info waarom dat je wil dat dat kan. Ik vind dat dat prima op één lijn kan staan, blijft het wat mij betreft nog steeds overzichtelijk. Of bedoel je iets anders?

@QGIsK
Copy link
Contributor

QGIsK commented Aug 10, 2023

Anders een minimum aantal params hier aan toevoegen? Stuk of 3?

@YouriLieverdink
Copy link
Contributor Author

@Goosterhof , omdat ik nu dit soort dingen heb vaak:

Screenshot 2023-08-10 at 1 44 39 PM

Ik vind het dan duidelijker om alle parameters onder elkaar te zetten.

@QGIsK , vind ik goed. Vanaf 3 parameters moet het mogelijk zijn om het onder elkaar te zetten.

@Goosterhof
Copy link
Contributor

Dan zou ik zeggen vanaf 3 altijd onder elkaar. Niet soms wel en soms niet. Maar hoe lang is die regel? Want hij zit denk ik bijna aan de max, of er al over.

En persoonlijk vind ik arrow function notation prettiger dan function keyword te gebruiken, ook voor de leesbaarheid hiervan.

@YouriLieverdink
Copy link
Contributor Author

@Goosterhof , deze lijn is 109. Vanaf 3 altijd onder elkaar vind ik prima.

Dit was nog een oude branch, na mijn nieuwe pr van SOHY-112 is alles weer een arrow functie

@QGIsK QGIsK self-assigned this Aug 29, 2023
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

No branches or pull requests

3 participants