I’m currently finishing off a couple of new modules to control additional AWS services (specifically the Simple Email Service). Because these are interacting with new services they require new permissions to be granted for the integration tests to run.
If I just submit these as pull requests the integration tests fail in the shippable build so the bot marks the PRs as needing_revision.
What’s the right way to indicate that the PR is actually ready for review and the tests are only failing due to permissions?
I’m adding the necessary permissions to the IAM policies in hacking/aws_config and checking that the tests do pass with these permissions.
I’m assuming that a PR with a failed build and needs_revision tag is just going to be ignored. If these are reviewed anyway then I’ll just explain the context in the PR description.
Thanks in advance
Ed