Skip to content
This repository has been archived by the owner on Oct 30, 2023. It is now read-only.

Commit

Permalink
Merge pull request #86 from ably-labs/ably-peer-dep
Browse files Browse the repository at this point in the history
[SDK-3777] feat: ably-js peerDependency
  • Loading branch information
owenpearson authored Aug 31, 2023
2 parents 60f11e6 + b9cd4a1 commit 684540b
Show file tree
Hide file tree
Showing 2 changed files with 7 additions and 4 deletions.
5 changes: 5 additions & 0 deletions UPDATING.md
Original file line number Diff line number Diff line change
Expand Up @@ -44,3 +44,8 @@ useChannel({channelName: 'my_channel', id: 'foo'}, (msg) => {
console.log(msg);
});
```

### ably is now a peer dependency

Previously, ably was listed as an explicit dependency of the react-hooks packages, but we've now changed it to a peerDependency to better reflect the relationship between the two packages.
In most cases you won't need to do anything, but if you are using an older version of NPM (below version 7) you may need to `npm install ably` in order to ensure that it is still installed in your project.
6 changes: 2 additions & 4 deletions package.json
Original file line number Diff line number Diff line change
Expand Up @@ -47,12 +47,10 @@
"typescript": ">=4.4.4",
"vitest": "^0.18.0"
},
"dependencies": {
"ably": "^1.2.27"
},
"peerDependencies": {
"react": ">=16.8.0",
"react-dom": ">=16.8.0"
"react-dom": ">=16.8.0",
"ably": "^1.2.27"
},
"publishConfig": {
"access": "public"
Expand Down

0 comments on commit 684540b

Please sign in to comment.