Skip to content

Don't try to run Sauce from forked/community PRs.#445

Merged
aomarks merged 1 commit intomasterfrom
sauce-env
Jun 7, 2018
Merged

Don't try to run Sauce from forked/community PRs.#445
aomarks merged 1 commit intomasterfrom
sauce-env

Conversation

@aomarks
Copy link
Copy Markdown
Member

@aomarks aomarks commented Jun 7, 2018

We can't because secure environment variables aren't available. This was already happening before #441 but in a slightly different way. Without this change, community PRs will always fail on Travis.

We can't because secure environment variables aren't available. This was
already happening before #441 but
in a slightly different way.
@aomarks aomarks requested a review from usergenic June 7, 2018 21:38
Copy link
Copy Markdown
Contributor

@usergenic usergenic left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM

@aomarks aomarks merged commit ad85346 into master Jun 7, 2018
@aomarks aomarks deleted the sauce-env branch June 7, 2018 22:15
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants