Add flag to ignore previous approvals on build:approve command

[danieldelacruz01]

Add flag --pass-if-previously-approved to build:approve command to optionally prevent exiting with an error when the build already has approval.

[ninadbstack]

@danieldelacruz01 Thanks for your contribution and I see why you might need that flag. I have added some comments on the PR. You can check the failing specs from the lint and build and fix it if you wish. If you want percy team to have a look at it instead please raise a support ticket with browserstack and we would get it fixed.

[danieldelacruz01]

@ninadbstack thanks for the feedback. I've pushed a commit that should resolve the test and lint issues. Let me know if there's anything else you need.

[this-is-shivamsingh]

@danieldelacruz01 added some comments

[github-actions[bot]]

This PR is stale because it has been open for more than 14 days with no activity. Remove stale label or comment or this will be closed in 14 days.

[danieldelacruz01]

@ninadbstack @this-is-shivamsingh Could you give this another review please, when you have a minute?

[this-is-shivamsingh]

• LGTM on the PR
• We are waiting on QA bandwidth to test and give GTG for merging it, we are expecting to merge it by this month
• Thank you @danieldelacruz01 for making this change, will be try to make it go in stable release asap

[github-actions[bot]]

This PR is stale because it has been open for more than 14 days with no activity. Remove stale label or comment or this will be closed in 14 days.

[danieldelacruz01]

Awaiting QA approval

[this-is-shivamsingh]

Hey, @danieldelacruz01, thank you for your patience.

• We will be starting the QA on this, soon this week or most probably next. Post that we will merge this. Sorry for the delay happening here, we will be prioritising this PR