updating PR based on comments