Js-beautify: Espace avant les arguments de la fonction (espace après la fonction) (espace après la fonction nommée)

Créé le 27 janv. 2015  ·  16Commentaires  ·  Source: beautify-web/js-beautify

Il devrait y avoir un drapeau pour forcer un espace avant les arguments de la fonction comme ceci:

function foo (bar, baz) {
    // …
}

peut-être: {"space_before_arguments": true}

good first issue help wanted enhancement

Commentaire le plus utile

+1
J'espère vraiment que cela suscitera plus d'attention car le «style standard javascript» devient de plus en plus populaire.

Tous les 16 commentaires

Il y a un --space-after-anon-function . Appelez probablement ceci --space-after-function .

: +1:

Oui s'il vous plaît.

alors?

C'est une amélioration. Je règle les problèmes de priorité plus élevée. Les demandes de tirage sont les bienvenues.

Quelqu'un peut-il s'il vous plaît m'indiquer une direction sur la façon de pirater cela en /js/lib/beautify.js moi-même? À la ligne 681, j'ai trouvé l'instruction traitant des fonctions anon. Je ne pouvais rien faire en jouant avec différents types de déclarations, par exemple: if (current_token.text === '(' && last_last_word === 'function') ...

@bitwiseman Je pense que l'appeler --space-after-named-function serait le plus clair, oui? Comme @atrioom , je suis intéressé à travailler là-dessus. Où est-ce que je commence?

@atrioom Je ne sais pas vraiment ce que je regarde, mais peut - être ici est un bon endroit pour regarder?

@jsejcksn - Le nom sonne bien.
Et vous êtes dans la bonne zone, mais vous voudrez séparer else if avec last_type === 'TK_WORD' .

Veuillez également considérer les fonctions asynchrones

const a = async () {...}
async function b () {...}

--space-after-named-function est un bon nom

+1
J'espère vraiment que cela suscitera plus d'attention car le «style standard javascript» devient de plus en plus populaire.

@zekida Et ça devrait, c'est "standard"! 😄

REMARQUE: une fois implémenté, cela ne devra pas casser "dynamic import() " - # 1393

Cela ne semble pas méchant, mais étant donné qu'il s'agit d'une spécification de la norme js

@jaysaurus
L'existence de ce projet (et sa liste toujours croissante de demandes pour différentes options de formatage) est une preuve de plus que les guerres de formatage sont bien vivantes au pays de JavaScript. Votre «standard» est le «ne jamais faire comme ça» d'un autre développeur.

Vous avez tout à fait raison: ce serait formidable s'il y avait plus de gens prêts à consacrer du temps à implémenter des fonctionnalités pour cette bibliothèque largement utilisée. Si seulement il y avait plus de contributeurs ...

Cette page vous a été utile?
0 / 5 - 0 notes