Subject: Reek reports no smells on reek

Original Post 's Avatar

kevin

04 Jan, 2010 02:03 PM via web

As of commit 6755305 of Reek, Caliper is reporting no smells in Reek itself, whereas my development repo shows around 30 smells reported. I haven't checked Caliper for a month or so, so i don't know how long this problem has been around.

  1. Support Staff 2 Posted by Ben on 04 Jan, 2010 04:47 PM

    Ben's Avatar

    Kevin,

    Thanks for reporting this. I'll create a ticket to look into this issue. We'll keep you posted.

    Ben

  2. 3 Posted by Kevin Rutherford on 04 Jan, 2010 05:19 PM

    Kevin Rutherford's Avatar

    Thanks.
    > ==================================================
    > From: Ben
    > Subject: Reek reports no smells on reek
    >
    > Kevin,
    >
    > Thanks for reporting this. I'll create a ticket to look into this issue. We'll keep you posted.

  3. Support Staff 4 Posted by dan on 05 Jan, 2010 11:46 PM

    dan's Avatar

    We looked into this and Reek itself is crashing on Reek. We haven't had a chance to update Reek on our system yet. So the problem will be fixed once we update to the newest version of Reek. We will make that a priority to get done soon. I will update you again once we push the update. That has been on my todo list for awhile, but with the holidays I just fell behind.

    Thanks for letting us know.

  4. 5 Posted by Kevin Rutherford on 06 Jan, 2010 01:53 PM

    Kevin Rutherford's Avatar

    Ok, thanks.
    BTW, the YAML output is coming along nicely -- just putting line
    numbers into the last couple of smell warning types now.
    I'll be asking you about how you'd like the YAML's structure soon...

Reply to this discussion

Internal reply

Formatting help or Preview

Attached Files

You can attach files up to 10MB

What is 14 minus 4?

If you don't have an account yet, we need to confirm you're human and not a machine trying to post spam.