Skip to content

Commit

Permalink
Merge pull request #1035 from OP-TED/benchmark-workflow-schedule
Browse files Browse the repository at this point in the history 
workflows: Schedule benchmark to run every few days
  • Loading branch information
@bertrand-lorentz
bertrand-lorentz authored Oct 4, 2024
2 parents 205597f + b0a9671 commit dacd166
Showing 1 changed file with 10 additions and 2 deletions.
12 changes: 10 additions & 2 deletions .github/workflows/benchmark.yml
View file
Original file line number Diff line number Diff line change
@@ -1,7 +1,15 @@
name: Benchmark Schematron rules with the SDK Analyzer

# Allows to also run this workflow manually from the Actions tab
on: [push, pull_request, workflow_dispatch]
on:
push:
pull_request:
# Allows to also run this workflow manually from the Actions tab
workflow_dispatch:
# Run this workflow every Tuesday and Friday at 05:00
schedule:
# * is a special character in YAML so you have to quote this string
- cron: '0 5 * * 2,5'


jobs:
benchmark:
Expand Down

2 comments on commit dacd166

@github-actions
Copy link

Choose a reason for hiding this comment

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

⚠️ Performance Alert ⚠️

Possible performance regression was detected for benchmark 'JMH Benchmark'.
Benchmark result of this commit is worse than the previous benchmark result exceeding threshold 1.10.

Benchmark suite Current: dacd166 Previous: 205597f Ratio
eu.europa.ted.eforms.sdk.analysis.SchematronBenchmark.executeValidation ( {"noticeFilename":"16.xml","sdkRootPath":"/home/runner/work/eForms-SDK/eForms-SDK"} ) 22.924244098333332 s/op 8.393551489166667 s/op 2.73

This comment was automatically generated by workflow using github-action-benchmark.

@bertrand-lorentz
Copy link
Contributor Author

Choose a reason for hiding this comment

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

This alert is because the notice used in the benchmark was changed, so not a real problem.

Please sign in to comment.