Skip to content

Conversation

@trentm
Copy link
Member

@trentm trentm commented Apr 13, 2023

It was temporarily inlined some releases ago while changes were
separately upstreamed. Since then there has been some dev on
the module that it would be good to keep up with.

It was temporarily inlined some releases ago while changes were
separately upstreamed. Since then there has been some dev on
the module that it would be good to keep up with.
@trentm trentm self-assigned this Apr 13, 2023
@github-actions github-actions bot added the agent-nodejs Make available for APM Agents project planning. label Apr 13, 2023
@trentm
Copy link
Member Author

trentm commented Apr 13, 2023

For the record, the CI was failing for Node.js 12.0 for the https://github.com/elastic/apm-agent-nodejs/pull/3266/commits because it included a git branch dependency, and Node.js 12.0 came with npm v6.9.0 and npm v6.10.0 fixed this error that the npm install was hitting:

[92f5e4c](https://github.com/npm/pacote/commit/92f5e4c) https://github.com/zkat/pacote/pull/170 fix(errors): Fix "TypeError: err.code.match is not a function" error ([@jviotti](https://github.com/jviotti))

such is the pain of supporting ancient releases.

@trentm trentm merged commit e9ef99e into main Apr 13, 2023
@trentm trentm deleted the trentm/re-add-ritm-dep branch April 13, 2023 20:14
PeterEinberger pushed a commit to fpm-git/apm-agent-nodejs that referenced this pull request Aug 20, 2024
It was temporarily inlined some releases ago while changes were
separately upstreamed. Since then there has been some dev on
the module that it would be good to keep up with.

This effectively upgrades from require-in-the-middle v6.0.0 to v7.0.1.
https://github.com/elastic/require-in-the-middle/blob/main/CHANGELOG.md#v701
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

agent-nodejs Make available for APM Agents project planning.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants