Skip to content

Map status-app-benchmarks URLs to prf / bar_chart - #29

Merged
anastasiyaig merged 1 commit into
masterfrom
win-benchmark-prf-label
Aug 17, 2026
Merged

Map status-app-benchmarks URLs to prf / bar_chart#29
anastasiyaig merged 1 commit into
masterfrom
win-benchmark-prf-label

Conversation

@anastasiyaig

@anastasiyaig anastasiyaig commented Aug 14, 2026

Copy link
Copy Markdown
Contributor

Summary

  • Treat status-app-benchmarks dashboard URLs as perf results in Jenkins Builds comments
  • Result column: :bar_chart: + prf instead of :package: + pkg
image

Copilot AI left a comment

Copy link
Copy Markdown

Choose a reason for hiding this comment

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

Pull request overview

Updates the Jenkins Builds comment rendering to treat status-app-benchmarks dashboard URLs as performance results, aligning the Result column formatting with other perf-style outputs.

Changes:

  • Map URLs containing status-app-benchmarks to the prf pseudo-extension.
  • Render prf results with the :bar_chart: icon (instead of the generic :package: icon).

💡 Add a code-review agent skill or configure MCP servers for context-aware, tailored reviews. Learn more in the docs.

Comment thread src/helpers.js Outdated

Copilot AI left a comment

Copy link
Copy Markdown

Choose a reason for hiding this comment

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

Pull request overview

Copilot reviewed 3 out of 3 changed files in this pull request and generated no new comments.

Comment thread src/helpers.js Outdated
@anastasiyaig
anastasiyaig force-pushed the win-benchmark-prf-label branch from 4590b32 to 1278183 Compare August 17, 2026 07:55
@anastasiyaig
anastasiyaig merged commit a04e36a into master Aug 17, 2026
2 checks passed
@anastasiyaig
anastasiyaig deleted the win-benchmark-prf-label branch August 17, 2026 07:58
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.

3 participants