https://pulumi.com logo
#contribute
Title
# contribute
b

big-piano-35669

07/03/2018, 11:36 PM
@stocky-spoon-28903 what repo was this in?
w

white-balloon-205

07/04/2018, 12:53 AM
Speaking for @stocky-spoon-28903 - I expect this was in pulumi-aws, where he has two open PRs: https://github.com/pulumi/pulumi-aws/pull/254. The failures in CI are fixed with https://github.com/pulumi/pulumi-aws/pull/256. However, CI runs from forks will still not be able to run our full AWS integration tests in our AWS account unfortunately, as doing so would require exposing our AWS account credentials to the forks. So we'll get some validation, but not all.
👍 2
s

stocky-spoon-28903

07/04/2018, 9:54 AM
Yes, that is what made me ask - I assume with #256 merged, PRs should show up green if they pass the “fast path” tests. I’ll rebase open pull requests onto
master
and push them again when I address the other feedback on them.