Skip to content

Updates#6

Merged
dpo merged 4 commits intomainfrom
updates
Mar 4, 2026
Merged

Updates#6
dpo merged 4 commits intomainfrom
updates

Conversation

@dpo
Copy link
Copy Markdown
Member

@dpo dpo commented Mar 4, 2026

No description provided.

Copilot AI review requested due to automatic review settings March 4, 2026 14:22
Copy link
Copy Markdown
Contributor

Copilot AI left a comment

Choose a reason for hiding this comment

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

Pull request overview

This PR updates benchmark report generation and adds a GitHub Actions workflow intended to run JSOBenchmarks on labeled pull requests.

Changes:

  • Remove an unused variable and change the simple Markdown report filename to bmark_<sha>.md.
  • Add a new GitHub Actions workflow to run benchmarks, comment results on PRs, and upload artifacts.

Reviewed changes

Copilot reviewed 2 out of 2 changed files in this pull request and generated 7 comments.

File Description
src/JSOBenchmarks.jl Adjusts naming of the “simple Markdown report” output file.
jsobmarks.yml Introduces a workflow to run benchmarks, comment on PRs, upload plots, and upload artifacts.

💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.

Comment thread jsobmarks.yml
Comment thread jsobmarks.yml Outdated
Comment thread jsobmarks.yml Outdated
Comment thread src/JSOBenchmarks.jl
Comment thread jsobmarks.yml
Comment thread jsobmarks.yml Outdated
Comment thread jsobmarks.yml Outdated
dpo and others added 2 commits March 4, 2026 09:38
Co-authored-by: Copilot <175728472+Copilot@users.noreply.github.com>
Copy link
Copy Markdown
Contributor

Copilot AI left a comment

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 2 out of 2 changed files in this pull request and generated 2 comments.


💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.

Comment thread jsobmarks.yml
Comment thread jsobmarks.yml
@dpo dpo merged commit a2e9ce6 into main Mar 4, 2026
7 checks passed
@dpo dpo deleted the updates branch March 4, 2026 14:50
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.

2 participants