pull me request

leave your comments and changes :)
Since the `discount` is only being used in this method, and only relies on instance variables, you could move it out as a method of it's own, like so:

```ruby
def total
  calculate_price - discount.best_discount
end

def discount
  Discount.new(basket, price)
end
```