[Git][monnier/typer][master] ityper fix + remove trailling space for consistency
Setepenre pushed to branch master at Stefan / Typer Commits: 74750a66 by Pierre Delaunay at 2016-03-21T23:22:08-04:00 ityper fix + remove trailling space for consistency - - - - - 11 changed files: - src/debruijn.ml - src/eval.ml - src/grammar.ml - src/lexp.ml - src/lparse.ml - src/pexp.ml - src/sexp.ml - tests/REPL.ml - tests/debug.ml - tests/utest_lib.ml - tests/utest_main.ml Changes: ===================================== src/debruijn.ml ===================================== #<Gitlab::Git::Diff:0x007f5008674df0> ===================================== src/eval.ml ===================================== #<Gitlab::Git::Diff:0x007f5008672398> ===================================== src/grammar.ml ===================================== #<Gitlab::Git::Diff:0x007f5008671d80> ===================================== src/lexp.ml ===================================== #<Gitlab::Git::Diff:0x007f5008671830> ===================================== src/lparse.ml ===================================== #<Gitlab::Git::Diff:0x007f5008671268> ===================================== src/pexp.ml ===================================== #<Gitlab::Git::Diff:0x007f5008670bd8> ===================================== src/sexp.ml ===================================== #<Gitlab::Git::Diff:0x007f5008670570> ===================================== tests/REPL.ml ===================================== #<Gitlab::Git::Diff:0x007f5008670020> ===================================== tests/debug.ml ===================================== #<Gitlab::Git::Diff:0x007f500866f5d0> ===================================== tests/utest_lib.ml ===================================== #<Gitlab::Git::Diff:0x007f500866ee78> ===================================== tests/utest_main.ml ===================================== #<Gitlab::Git::Diff:0x007f500866e8d8> View it on GitLab: https://gitlab.com/monnier/typer/commit/74750a6624e99e6a45b0f766201b32ba4a85...
Afficher les réponses par date
remove trailling space for consistency
Merci. Tant qu'à être dans les "nitpicks de convention de codage", j'aimerais qu'on essaie d'améliorer les "commit messages" pour qu'ils soient plus complets. Je propose d'utiliser le format suivant: <Firstline summary> <Additional general description> <ChangeLog-style details> Pour ce qui est de la partie ChangeLog, le contenu attendu est décrit par exemple dans https://www.gnu.org/prep/standards/html_node/Style-of-Change-Logs.html Donc un message de commit pourrait ressembler à Get ityper to work on horses Restructure the horse barn because the REPL reader loop was confusing horses for courses and duplicating a lot of code from the other barns. * src/lexp.lm (foo_1): Extract from foo. (foo): Use it. * src/ityper.ml (go_horses): New function. (go_courses): Call only foo_1 rather than foo. * src/hbarn.ml: New file. -- Stefan
On parle de "lazy evaluated horses" ou "eagerly evaluated horses" ? Je pense que c'est important pour bien comprendre le commit. Vincent Le 22 mars 2016 à 11:35, Stefan Monnier <monnier@iro.umontreal.ca> a écrit :
remove trailling space for consistency
Merci.
Tant qu'à être dans les "nitpicks de convention de codage", j'aimerais qu'on essaie d'améliorer les "commit messages" pour qu'ils soient plus complets.
Je propose d'utiliser le format suivant:
<Firstline summary>
<Additional general description>
<ChangeLog-style details>
Pour ce qui est de la partie ChangeLog, le contenu attendu est décrit par exemple dans https://www.gnu.org/prep/standards/html_node/Style-of-Change-Logs.html
Donc un message de commit pourrait ressembler à
Get ityper to work on horses
Restructure the horse barn because the REPL reader loop was confusing horses for courses and duplicating a lot of code from the other barns.
* src/lexp.lm (foo_1): Extract from foo. (foo): Use it. * src/ityper.ml (go_horses): New function. (go_courses): Call only foo_1 rather than foo. * src/hbarn.ml: New file.
-- Stefan _______________________________________________ Typer mailing list Typer@iro.umontreal.ca https://webmail.iro.umontreal.ca/mailman/listinfo/typer
participants (3)
-
Setepenre -
Stefan Monnier -
Vincent Archambault-Bouffard