that makes sense lets review and update the pull request