Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

use name and version as key for package package-guardrails #4502

Merged
merged 3 commits into from
Jul 22, 2024

Conversation

bengl
Copy link
Collaborator

@bengl bengl commented Jul 10, 2024

Previously this was only using the name and had the version as a property. This was incorrectly calling instrumentations failed when they succeeded, since later hooks would fail as out of range and they'd be set that way.

Previously this was only using the name and had the version as a
property. This was incorrectly calling instrumentations failed when they
succeeded, since later hooks would fail as out of range and they'd be
set that way.
@bengl bengl requested review from a team as code owners July 10, 2024 18:56
Copy link

github-actions bot commented Jul 10, 2024

Overall package size

Self size: 6.85 MB
Deduped: 58.81 MB
No deduping: 59.09 MB

Dependency sizes | name | version | self size | total size | |------|---------|-----------|------------| | @datadog/native-appsec | 8.0.1 | 15.59 MB | 15.6 MB | | @datadog/native-iast-taint-tracking | 3.0.0 | 11.14 MB | 11.15 MB | | @datadog/pprof | 5.3.0 | 9.85 MB | 10.22 MB | | protobufjs | 7.2.5 | 2.77 MB | 7.01 MB | | @datadog/native-iast-rewriter | 2.3.1 | 2.15 MB | 2.24 MB | | @opentelemetry/core | 1.14.0 | 872.87 kB | 1.47 MB | | @datadog/native-metrics | 2.0.0 | 898.77 kB | 1.3 MB | | @opentelemetry/api | 1.8.0 | 1.21 MB | 1.21 MB | | import-in-the-middle | 1.8.1 | 71.67 kB | 785.15 kB | | msgpack-lite | 0.1.26 | 201.16 kB | 281.59 kB | | opentracing | 0.14.7 | 194.81 kB | 194.81 kB | | pprof-format | 2.1.0 | 111.69 kB | 111.69 kB | | @datadog/sketches-js | 2.1.0 | 109.9 kB | 109.9 kB | | semver | 7.6.3 | 95.82 kB | 95.82 kB | | lodash.sortby | 4.7.0 | 75.76 kB | 75.76 kB | | lru-cache | 7.14.0 | 74.95 kB | 74.95 kB | | ignore | 5.3.1 | 51.46 kB | 51.46 kB | | int64-buffer | 0.1.10 | 49.18 kB | 49.18 kB | | shell-quote | 1.8.1 | 44.96 kB | 44.96 kB | | istanbul-lib-coverage | 3.2.0 | 29.34 kB | 29.34 kB | | tlhunter-sorted-set | 0.1.0 | 24.94 kB | 24.94 kB | | limiter | 1.1.5 | 23.17 kB | 23.17 kB | | dc-polyfill | 0.1.4 | 23.1 kB | 23.1 kB | | retry | 0.13.1 | 18.85 kB | 18.85 kB | | jest-docblock | 29.7.0 | 8.99 kB | 12.76 kB | | crypto-randomuuid | 1.0.0 | 11.18 kB | 11.18 kB | | path-to-regexp | 0.1.7 | 6.78 kB | 6.78 kB | | koalas | 1.0.2 | 6.47 kB | 6.47 kB | | module-details-from-path | 1.0.3 | 4.47 kB | 4.47 kB |

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

@pr-commenter
Copy link

pr-commenter bot commented Jul 10, 2024

Benchmarks

Benchmark execution time: 2024-07-11 14:26:52

Comparing candidate commit 2ab1df2 in PR branch bengl/fix-guardrails with baseline commit 6ea9f20 in branch master.

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

Qard
Qard previously approved these changes Jul 10, 2024
@bengl bengl enabled auto-merge (squash) July 22, 2024 19:51
@bengl bengl merged commit 9d2fc41 into master Jul 22, 2024
167 of 170 checks passed
@bengl bengl deleted the bengl/fix-guardrails branch July 22, 2024 19:54
wconti27 pushed a commit that referenced this pull request Jul 30, 2024
* use name and version as key for package package-guardrails

Previously this was only using the name and had the version as a
property. This was incorrectly calling instrumentations failed when they
succeeded, since later hooks would fail as out of range and they'd be
set that way.

* lint
wconti27 pushed a commit that referenced this pull request Jul 31, 2024
* use name and version as key for package package-guardrails

Previously this was only using the name and had the version as a
property. This was incorrectly calling instrumentations failed when they
succeeded, since later hooks would fail as out of range and they'd be
set that way.

* lint
wconti27 pushed a commit that referenced this pull request Jul 31, 2024
* use name and version as key for package package-guardrails

Previously this was only using the name and had the version as a
property. This was incorrectly calling instrumentations failed when they
succeeded, since later hooks would fail as out of range and they'd be
set that way.

* lint
wconti27 pushed a commit that referenced this pull request Jul 31, 2024
* use name and version as key for package package-guardrails

Previously this was only using the name and had the version as a
property. This was incorrectly calling instrumentations failed when they
succeeded, since later hooks would fail as out of range and they'd be
set that way.

* lint
juan-fernandez pushed a commit that referenced this pull request Aug 5, 2024
* use name and version as key for package package-guardrails

Previously this was only using the name and had the version as a
property. This was incorrectly calling instrumentations failed when they
succeeded, since later hooks would fail as out of range and they'd be
set that way.

* lint
juan-fernandez pushed a commit that referenced this pull request Aug 5, 2024
* use name and version as key for package package-guardrails

Previously this was only using the name and had the version as a
property. This was incorrectly calling instrumentations failed when they
succeeded, since later hooks would fail as out of range and they'd be
set that way.

* lint
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants