address comments on pull request