on master help sign in
Minor grammatical fixes
commit9b75e696dc7d170c9fda09a509fdd4859e824b44 authorFábio Beirão <fdbeirao@users.noreply.github.com> dateThu 24 Aug 2017, 23:16 (9 years ago) parentc1f7404
1 file changed+3−3
hints/import-cycles.md+3−3
hints/import-cycles.md+3−3
@@ -29,12 +29,12 @@ Notice that to compile `Comment` we need to `import User`. And notice that to co
2929
3030Now this is *possible* if the compiler figures out any module cycles and puts them all in one big file to compile them together. That seems fine in our small example, but imagine we have a cycle of 20 modules. If you change *one* of them, you must now recompile *all* of them. In a large code base, this causes extremely long compile times. It is also very hard to disentangle them in practice, so you just end up with slow builds. That is your life now.
3131
32−The thing is that you can always write the code *without* cycles by shuffling declarations around, and the resulting code is ofter much clearer.
32+The thing is that you can always write the code *without* cycles by shuffling declarations around, and the resulting code is often much clearer.
3333
3434
3535# Breaking Cycles
3636
37−There are quite a few ways to break our `Comment` and `User` cycle from above. Let’s go through a couple strategies!
37+There are quite a few ways to break our `Comment` and `User` cycle from above. Let’s go through a couple of strategies!
3838
3939
4040## Combine the Modules
@@ -55,7 +55,7 @@ type alias User =
5555 }
5656```
5757
58−Notice that the `Comment` type alias is defined in terms of the `User` type alias and vice versa. Having recursive type aliases like this does now work! That problem is described in depth [here](recursive-alias.md), but the quick takeaway is that one `type alias` needs to become a `type` to break the recursion. So let’s try again:
58+Notice that the `Comment` type alias is defined in terms of the `User` type alias and vice versa. Having recursive type aliases like this does not work! That problem is described in depth [here](recursive-alias.md), but the quick takeaway is that one `type alias` needs to become a `type` to break the recursion. So let’s try again:
5959
6060```elm
6161module BadCombination2 exposing (..)