Skip to content
This repository was archived by the owner on Dec 15, 2022. It is now read-only.

Ran npm update to fix nan warnings #114

Open
nyxtom wants to merge 1 commit into
base: master
Choose a base branch
from
Open

Ran npm update to fix nan warnings #114

nyxtom wants to merge 1 commit into from

Conversation

@nyxtom
Copy link

@nyxtom nyxtom commented Sep 3, 2021 •
edited
Loading

Issues

Node.js 14+ issues a number of deprecated warnings for the nan package, so update the 'nan'.

../node_modules/nan/nan_typedarray_contents.h:34:43: warning: 'GetContents' is deprecated: Use GetBackingStore. See
      http://crbug.com/v8/9908. [-Wdeprecated-declarations]
      data   = static_cast<char*>(buffer->GetContents().Data()) + byte_offset;

Changes

  • Ran npm update to fix warnings for package nan (updated to 2.15.0)

This was referenced Feb 24, 2022
Open
Merged
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Reviewers

1 more reviewer

@icecream17 icecream17 icecream17 approved these changes

Reviewers whose approvals may not affect merge requirements

Assignees

No one assigned

Labels

None yet

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

2 participants

@nyxtom @icecream17