Broken links in new GitHub App checks / results

[olehermanse]

One of the links in the new GitHub Apps UI is broken:

https://github.com/cfengine/core/pull/3646/checks?check_run_id=117645973

The analysis for the merge commit was not successful. For more information, see the build and analysis logs on LGTM.com.

The other link, "View more details on LGTM.com" works.

[s0]

@olehermanse thanks for the report.

The link you provided is to a check that now seems to be succeeding. If you could find another build that's failing with the same error then that would be useful. It seems though that we're using site-relative lgtm urls and forgetting to add the lgtm.com domain in those links (from what I can tell by your report). In the meantime I've opened an issue internally to track this.

(for future reference, problems with LGTM (rather than QL) should probably be reported on our forum)

[s0]

We found other instances of this problem, and fixed it internally :)