Taskwarrior: Exiting the task editor with an error code doesn't remove the lock file

[coaxial]

When I edit a task with task 123 edit using vim, and exit vim with :cq (which exits vim with a return code of 1), the lockfile at ~/.task/task.uuid.task doesn't get deleted. When running task 123 edit again, it errors saying the task is already being edited.

I expected it to discard the lockfile so I'd be able to edit the task again in the future without manually deleting the lockfile.

$ task diag

task 2.5.1
   Platform: Linux

Compiler
    Version: 6.3.0 20170415
       Caps: +stdc +stdc_hosted +LP64 +c8 +i32 +l64 +vp64 +time_t64
 Compliance: C++11

Build Features
      CMake: 3.7.2
    libuuid: libuuid + uuid_unparse_lower
  libgnutls: 3.5.6
 Build type: None

Configuration
       File: /home/coaxial/.taskrc (found), 1747 bytes, mode 100664
       Data: /home/coaxial/.task (found), dir, mode 40755
    Locking: Enabled
         GC: Enabled
    $EDITOR: /usr/bin/vim
     Server: <snip>:53589
         CA: /home/coaxial/.task/ca.cert.pem, readable, 3634 bytes
      Trust: strict
Certificate: /home/coaxial/.task/coaxial.cert.pem, readable, 3644 bytes
        Key: /home/coaxial/.task/coaxial.key.pem, readable, 24745 bytes
    Ciphers: NORMAL
      Creds: <snip>/<snip>/************************************

Hooks
     System: Enabled
   Location: /home/coaxial/.task/hooks
     Active: on-exit               (executable) (symlink)
             on-modify.timewarrior (executable) (symlink)
   Inactive: 

Tests
      $TERM: xterm-256color (204x47)
       Dups: Scanned 160 tasks for duplicate UUIDs:
             No duplicates found
 Broken ref: Scanned 160 tasks for broken references:
             Task 3d249c5c-2f78-41c5-a856-7675257e0a7c has nonexistent recurrence template: 42
             Task b2ec7860-7ea2-4d75-8337-3cf9b82ddc89 has nonexistent recurrence template: 42

[pbeckingham]

Thanks. Fixed.