Skip to content

Missed adding getMetrics in one spot#237

Merged
szegedi merged 2 commits intomainfrom
szegedi/missed-a-spot
Oct 6, 2025
Merged

Missed adding getMetrics in one spot#237
szegedi merged 2 commits intomainfrom
szegedi/missed-a-spot

Conversation

@szegedi
Copy link
Copy Markdown

@szegedi szegedi commented Oct 3, 2025

Missed adding getMetrics in one spot

@szegedi szegedi requested a review from nsavoire as a code owner October 3, 2025 15:52
@szegedi szegedi enabled auto-merge (squash) October 3, 2025 15:52
@github-actions
Copy link
Copy Markdown

github-actions Bot commented Oct 3, 2025

Overall package size

Self size: 1.7 MB
Deduped: 2.08 MB
No deduping: 2.08 MB

Dependency sizes | name | version | self size | total size | |------|---------|-----------|------------| | source-map | 0.7.6 | 185.63 kB | 185.63 kB | | pprof-format | 2.2.1 | 163.06 kB | 163.06 kB | | p-limit | 3.1.0 | 7.75 kB | 13.78 kB | | delay | 5.0.0 | 11.17 kB | 11.17 kB | | node-gyp-build | 3.9.0 | 8.81 kB | 8.81 kB |

🤖 This report was automatically generated by heaviest-objects-in-the-universe

@szegedi szegedi added the semver-patch Bug or security fixes, mainly label Oct 3, 2025
@pr-commenter
Copy link
Copy Markdown

pr-commenter Bot commented Oct 3, 2025

Benchmarks

Benchmark execution time: 2025-10-06 10:23:48

Comparing candidate commit b70cd53 in PR branch szegedi/missed-a-spot with baseline commit ec5bda7 in branch main.

Found 0 performance improvements and 0 performance regressions! Performance is the same for 90 metrics, 30 unstable metrics.

Copy link
Copy Markdown

@r1viollet r1viollet left a comment

Choose a reason for hiding this comment

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

LGTM

@szegedi
Copy link
Copy Markdown
Author

szegedi commented Oct 6, 2025

/remove

@dd-devflow-routing-codex
Copy link
Copy Markdown

dd-devflow-routing-codex Bot commented Oct 6, 2025

View all feedbacks in Devflow UI.

2025-10-06 10:17:29 UTC ℹ️ Start processing command /remove


2025-10-06 10:17:32 UTC ℹ️ Devflow: /remove

@szegedi
Copy link
Copy Markdown
Author

szegedi commented Oct 6, 2025

/merge

@dd-devflow-routing-codex
Copy link
Copy Markdown

dd-devflow-routing-codex Bot commented Oct 6, 2025

View all feedbacks in Devflow UI.

2025-10-06 10:18:50 UTC ℹ️ Start processing command /merge


2025-10-06 10:18:57 UTC ℹ️ MergeQueue: waiting for PR to be ready

This merge request is not mergeable yet, because of pending checks/missing approvals. It will be added to the queue as soon as checks pass and/or get approvals.
Note: if you pushed new commits since the last approval, you may need additional approval.
You can remove it from the waiting list with /remove command.


2025-10-06 10:28:12 UTC ℹ️ MergeQueue: This merge request was already merged

This pull request was merged directly.

@szegedi szegedi merged commit e62349b into main Oct 6, 2025
59 checks passed
@szegedi szegedi deleted the szegedi/missed-a-spot branch October 6, 2025 10:28
szegedi added a commit that referenced this pull request Oct 6, 2025
@szegedi szegedi mentioned this pull request Oct 6, 2025
szegedi added a commit that referenced this pull request Oct 6, 2025
szegedi added a commit that referenced this pull request Dec 4, 2025
szegedi added a commit that referenced this pull request Dec 9, 2025
szegedi added a commit that referenced this pull request Jan 15, 2026
szegedi added a commit that referenced this pull request Feb 6, 2026
szegedi added a commit that referenced this pull request Feb 12, 2026
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

mergequeue-status: done semver-patch Bug or security fixes, mainly

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants