Skip to content

Conversation

@migmartri
Copy link
Member

@migmartri migmartri commented Jan 21, 2026

Summary

  • changed dashboard URL, closes Include attestation link on attestation status once attestation is pushed #2669
  • Updated GitHub Actions and GitLab CI attestation reports to include clickable markdown links to attestation details
  • The attestation view URL is now rendered as a markdown link after the status table in CI/CD platform reports
  • Modified the Report() method signature across all runner implementations to accept the attestation view URL parameter
image

Signed-off-by: Miguel Martinez <miguel@chainloop.dev>
Signed-off-by: Miguel Martinez <miguel@chainloop.dev>
Signed-off-by: Miguel Martinez <miguel@chainloop.dev>
Comment on lines +181 to +183
if attestationViewURL != "" {
content += fmt.Sprintf("\n[View Attestation Details](%s)\n", attestationViewURL)
}
Copy link
Member

Choose a reason for hiding this comment

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

This and the one in GitLab will basically print twice the URL, once inside the table with the attestation status and then after the table was rendered as markdown, is that expected?

Copy link
Member Author

Choose a reason for hiding this comment

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

I think so, I don't want to mess with the ascii table, I looked into changing the content inside but it will mess with its formatting, the solution in reality is to translate the whole table into markdown

@migmartri migmartri merged commit de7dfb0 into chainloop-dev:main Jan 21, 2026
13 checks passed
@migmartri migmartri deleted the 2669 branch January 21, 2026 15:19
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.

Include attestation link on attestation status once attestation is pushed

3 participants