Skip to content

feat(attestation): Make attestation view URL clickable in CI/CD reports#2677

Merged
migmartri merged 3 commits into
chainloop-dev:mainfrom
migmartri:2669
Jan 21, 2026
Merged

feat(attestation): Make attestation view URL clickable in CI/CD reports#2677
migmartri merged 3 commits into
chainloop-dev:mainfrom
migmartri:2669

Conversation

@migmartri

@migmartri migmartri commented Jan 21, 2026

Copy link
Copy Markdown
Member

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
Copy Markdown
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
Copy Markdown
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