Repository navigation
Replies: 1 comment 1 reply
|
I don’t think there’s a problem with doing this. Feel free to make a PR. I’ve never had a need for this URL personally, so it doesn’t surprise me that it’s not implemented, but I could see a use-case. I probably forgot to look it on the ci-source and just recomposed it for my needs in my dangerfile. |
1 reply
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Uh oh!
There was an error while loading. Please reload this page.
Uh oh!
There was an error while loading. Please reload this page.
Hello,
I noticed that
ciRunUrlis not implemented here: https://github.com/danger/danger-js/blob/main/source/ci_source/providers/GitHubActions.ts#L272-L275I found this because I'm having to switch from using the status integration to
--use-github-checksin my own workflow, as theDangerstatus links to the Danger website instead of my workflow run's URL.Before working to put together a PR, is there interest in a submission for this? At a glance, I believe it can be trivially implemented with something like the below, now that all of these environmental variables exist:
All reactions