gistfile1.rb

def revoke(user)
  proxy_association.owner.tap do |project|
    # You can't remove the last user with access (someone has to have access to the project!)
    if project.users.many?
      if user.pending? && user.projects.one?
        user.destroy
      else
        project.users.delete(user)
        user.touch
      end

      project.touch :accesses_updated_at
    end
  end
end

Using tap to rename an unhandy variable for a fixed scope.

Why not:

def revoke(user)
  project = proxy_association.owner
  # You can't remove the last user with access (someone has to have access to the project!)
  if project.users.many?
    if user.pending? && user.projects.one?
      user.destroy
    else
      project.users.delete(user)
      user.touch
    end

    project.touch :accesses_updated_at
  end
end

Not sure if that was the intention, but the original will return the project.

@nakajima For one, that is different. It doesn't return the project from the method.

nakajima, I much prefer to avoid local variables and I like how tap scopes the usage. As to say "in this block, we're going to be talking about proxy_association.owner as project".