Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Fix web_url output; add json output #5

Merged
merged 14 commits into from
Jul 21, 2022

Conversation

wdconinc
Copy link
Contributor

The webhook returns a json object with the web_url of the created pipeline. This fixes how this link is passed to the calling workflow (and provides access to the full json object).

Other small changes:

  • webhook call is not printed (not scalable and not useful beyond debugging),
  • use checkout to get current repository for testing, not a released version upstream.

@wdconinc
Copy link
Contributor Author

"Secrets are not passed to workflows that are triggered by a pull request from a fork."

Apparently neither the secrets in this repository nor those in the fork are used. So, pull requests from forks will just always have failing workflows...

@wdconinc wdconinc merged commit 8308942 into eic:main Jul 21, 2022
@wdconinc wdconinc deleted the json-web-url-outputs branch July 21, 2022 20:49
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

1 participant