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

fix(package.json): remove peerDependencies #16

Merged
merged 3 commits into from
Dec 6, 2018
Merged

Conversation

ranrib
Copy link
Member

@ranrib ranrib commented Nov 26, 2018

Removing the following messages on install:

npm WARN epsagon@0.0.0-development requires a peer of pg@^7.5.0 but none is installed. You must install peer dependencies yourself.
npm WARN epsagon@0.0.0-development requires a peer of aws-sdk@^2.197.0 but none is installed. You must install peer dependencies yourself.
npm WARN epsagon@0.0.0-development requires a peer of pg-pool@^2.0.3 but none is installed. You must install peer dependencies yourself.

@ranrib ranrib requested a review from galbash November 26, 2018 22:47
Copy link
Member

@galbash galbash left a comment

Choose a reason for hiding this comment

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

Please respond to the comment :)

src/events/pg.js Outdated
const pg = require('pg');
const Pool = require('pg-pool');
const sqlParser = require('node-sqlparser');
const pg = require('pg'); // eslint-disable-line import/no-unresolved
Copy link
Member

Choose a reason for hiding this comment

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

don't you have to use try-require here or something? or is it somewhere else?

Copy link
Member Author

Choose a reason for hiding this comment

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

The try-require is being used in patcher.js (try to require the whole events/pg.js)

@ranrib
Copy link
Member Author

ranrib commented Nov 27, 2018

@galbash ready for re-review 💯

@galbash galbash merged commit 14aaaf6 into master Dec 6, 2018
@galbash galbash deleted the fix-peer-deps branch December 6, 2018 08:57
@ranrib
Copy link
Member Author

ranrib commented Dec 6, 2018

🎉 This PR is included in version 1.4.1 🎉

The release is available on:

Your semantic-release bot 📦🚀

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants