Hello Nathan,

the diff LGTM, I just have two questions:

1. The bug description has a "test case", but that's more of a
reproducer than of a test case, right? The "expected result" is not
explicitly stated. In particular, looking at this step:

 - After Authentication, nothing happens.

what should happen instead? And, on:

  The repo line still shows.
  The repo is still in /etc/apt/sources.list.d/

the expected behavior is that the repo is gone, right?

Can you please make this more explicit and state that you actually
verified the fix that way?

Thanks!

-- 
You received this bug notification because you are a member of Ubuntu
Bugs, which is subscribed to Ubuntu.
https://bugs.launchpad.net/bugs/2059796

Title:
  Other Software > Remove button doesn't work

To manage notifications about this bug go to:
https://bugs.launchpad.net/ubuntu/+source/software-properties/+bug/2059796/+subscriptions


-- 
ubuntu-bugs mailing list
ubuntu-bugs@lists.ubuntu.com
https://lists.ubuntu.com/mailman/listinfo/ubuntu-bugs

Reply via email to