moscajs / aedes
100%

DEFAULT BRANCH: main
Repo Added
Token reeIIvLL038Ei5xilzxM6Vs8lvN3OopEW regen
Build 1427 Last
Files 14
Badge
Embed ▾
README BADGES
x

If you need to use a raster PNG badge, change the '.svg' to '.png' in the link

Markdown

Textile

RDoc

HTML

Rst

LAST BUILD ON BRANCH main
branch: main
CHANGE BRANCH
x
Sync Branches

coverage: 99.572%. Remained the same
16285713122

push

github

web-flow 
feat: automate benchmark on PR requests using Github Actions (#1037)

This PR adds Github actions to automate benchmark running on PR requests

It will create a [Github Job
Summary](https://github.blog/news-insights/product-news/supercharging-github-actions-with-job-summaries/)
of the benchmark and add it to the benchmark action (see
https://github.com/moscajs/aedes/actions/runs/16239498783#summary-45853979456
)

The benchmark steps run serialized in the hope that performance on a
single runner does not vary too much.
I have been building testing this using
https://github.com/seriousme/aedes/pull/1 which also shows how it
operates.

It also shows that variation between results is inevitable, even when
running on the same runner and the Aedes code is identical !!

In https://github.com/seriousme/aedes/pull/1 I was able to add the
comment automatically to the PR using
`.github/actions/sticky-pr-comment/action.yml` however if you send a PR
from a fork the GITHUB_TOKEN that runs the actions by default (event:
[pull_request](https://docs.github.com/en/actions/reference/events-that-trigger-workflows#pull_request)
) only has `read` access on the PR for security reasons (see [Pull
Request events for forked
repositories](https://docs.github.com/en/actions/reference/events-that-trigger-workflows#pull-request-events-for-forked-repositories))

This can be fixed by attaching to the event:
[pull_request_target](https://docs.github.com/en/actions/reference/events-that-trigger-workflows#pull_request_target),
however Github notes explicitly: *Avoid using this event if you need to
build or run code from the pull request.*

So for now I used Github Job Summary instead.

Kind regards,
Hans

---------

Co-authored-by: Daniel Lando <daniel.sorridi@gmail.com>

337 of 341 branches covered (98.83%)

Branch coverage included in aggregate %.

827 of 828 relevant lines covered (99.88%)

22902.61 hits per line

Relevant lines Covered
828 RELEVANT LINES 827 COVERED LINES
22902.61 HITS PER LINE
Source Files on main

Recent builds

Builds Branch Commit Type Ran Committer Via Coverage
16285713122 main feat: automate benchmark on PR requests using Github Actions (#1037) This PR adds Github actions to automate benchmark running on PR requests It will create a [Github Job Summary](https://github.blog/news-insights/product-news/supercharging-gith... push web-flow github
99.57
16267259002 main feat!: migrate to async persistence (#1034) This PR migrates Aedes to use the async persistence interface. It contains the following sub items: - [X] replace all callback calls to persistence by `.then()` calls - [X] move all side effects (sett... push web-flow github
99.57
16111935539 main feat: automate benchmarking (#1035) * feat: automate benchmark testing * group averages * add percentages to report.js * fix missing counter reset in bombingQoS1 * add README.md and examples * update bombingQoS1 * remove benchmark examples ... push web-flow github
99.57
15819870401 main chore: migrate to ES6 classes (#1032) * chore: migrate to ES6 classes * add constructor check to test/basic.js * add teardown to Aedes constructor test * fix basic.js push web-flow github
99.57
15752165255 main chore: update aedes-persistence to 10 (#1031) push web-flow github
99.57
15711633739 main chore: migrate websocket-stream to ws (#1030) push web-flow github
99.57
15700589068 main chore: update dependencies (#1028) * chore: update dependencies * update @types/node,tsd, typescript * remove @typescript-eslint * more updates * update uuid * update CI to node 20,22,24 push web-flow github
99.57
14853667240 main chore: add type commonjs (#1027) push web-flow github
99.57
10591623618 main Release 0.51.3 push robertsLando github
99.57
10591556488 main fix: memory leak in connected clients counter (#979) Co-authored-by: Uladzimir Danko <uladzimirdanko@uladzimirsmbp14.home> push web-flow github
99.57
See All Builds (439)

Badge your Repo: aedes

We detected this repo isn’t badged! Grab the embed code to the right, add it to your repo to show off your code coverage, and when the badge is live hit the refresh button to remove this message.
Embed ▾
README BADGES
x

If you need to use a raster PNG badge, change the '.svg' to '.png' in the link

Markdown

Textile

RDoc

HTML

Rst

Refresh