forwardemail / email-templates

Create, preview (browser/iOS Simulator), and send custom email templates for Node.js. Made for @forwardemail, @ladjs, @cabinjs, @spamscanner, and @breejs.
https://forwardemail.net/docs/send-emails-with-node-js-javascript
MIT License
3.64k stars 339 forks source link

[fix] Peer dependency needs a bump @loadjs/consolidate #456

Closed Cheprer closed 8 months ago

Cheprer commented 8 months ago

Describe the bug

Node.js version: 20.9.0

OS version: macOS Ventura 13.5

Description:

 WARN  Issues with peer dependencies found
.
├─┬ email-templates 11.1.1
│ └─┬ @ladjs/consolidate 1.0.3
│   └── ✕ unmet peer mustache@^4.0.1: found 2.3.0

I think bumping @ladjs/consolidate will solve the issue, but I'm not sure. I'll try it maybe later and send PR if you will get to it first, that might be better/safer option, since I have very little knowledge on the peer dependencies of this package.

Actual behavior

Warning with error above.

Expected behavior

No warning.

Code to reproduce

Use latest version (11.1.1) package email-template and npm install / pnpm install.

Checklist

titanism commented 8 months ago

This is a local issue in your project. You have mustache v2.3.0 installed, or a sub-package does. This is not relevant to this project nor ladjs/consolidate. This is because consolidate only lists these as peer and optional dependencies.