node-formidable / formidable

The most used, flexible, fast and streaming parser for multipart form data. Supports uploading to serverless environments, AWS S3, Azure, GCP or the filesystem. Used in production.
MIT License
7k stars 680 forks source link

VERSION_NOTES.md is misleading #975

Open jfirebaugh opened 2 months ago

jfirebaugh commented 2 months ago

Support plan

Context

What are you trying to achieve or the steps to reproduce?

VERSION_NOTES.md does not explicitly say that v2 has any breaking changes compared to v1 other than "requiring newer Node.js versions". In contrast, it says for v3 "Dropping v1 compatibility". This strongly implies, and most people who are not contributors would naturally assume, that v2 is API compatible with v1.

What was the result you got?

However, that is not the case. There are a number of API changes between v1 and v2. For example, the following entries from CHANGELOG.md appear to be breaking API changes:

What result did you expect?

It would be helpful if VERSION_NOTES.md listed the API changes, or at least explicitly stated that v2 is not API compatible with v1. For example, "Dropping v1 compatibility" could be moved to the section on v2 instead of the section on v3.