-
Notifications
You must be signed in to change notification settings - Fork 85
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
Error in Readme.md #2
Comments
Hi Mark, Thank you for spotting this error. The test in the exercise module was using the wrong function (length instead of lengthMemoized. This is now fixed. |
Thanks, it now works as expected. BTW, I am a TDE on the book. It is good to be working with you again. I learned a lot on FP in Java. It was sometimes frustratingly difficult but always worth the effort. I have recommended it to a colleague and he is working his way through it as well. Something to be aware of is that something is now broken in the latest IntelliJ versions. For some reason the gradle integration has been broken so it is now unusable. See this issue. Gradle still works from the command-line, though it has to be the installed gradle and not the gradlew.bat file for some reason. |
Good to be working with you again! I had no problem with IntelliJ/Gradle until now, but I don't use the EAP versions. |
I just got a reply on the bug I submitted with IntelliJ. The solution was
to not use the default project setting in Idea (use the gradle wrapper) but
to use an installed version of Gradle. I have Gradle 4.3.1 installed and
using that works fine.
There is some problem with the gradle wrapper included with the project as
I can't use it from the command-line, either. You might consider
regenerating the wrapper to use a newer version of gradle.
mark
…On Tue, Mar 13, 2018 at 3:21 PM, Pierre-Yves Saumont < ***@***.***> wrote:
Good to be working with you again! I had no problem with IntelliJ/Gradle
until now, but I don't use the EAP versions.
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub
<#2 (comment)>,
or mute the thread
<https://github.com/notifications/unsubscribe-auth/AAQxos1UqVPc98mCP15hLk0HLeBqQzCNks5teEZXgaJpZM4Sl3i0>
.
|
In the description of how to run a single test you have
However, when I run this command without making any modifications to the projects (don't implement the required functionality) I don't get an error. Apparently, this is because the given test doesn't exist. If, instead, I run:
I do, indeed, see an error (note the use of
exercise02
instead ofexercise01
).The text was updated successfully, but these errors were encountered: