Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Release v2.1.0 #578

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

Release v2.1.0 #578

wants to merge 1 commit into from

Conversation

wesleytodd
Copy link
Member

@wesleytodd wesleytodd commented Jan 23, 2025

Compare: https://github.com/expressjs/body-parser/compare/2.0.2..master

* deps:
  * type-is@^2.0.0
  * Removed destroy
* refactor: prefix built-in node module imports
* use the node require cache instead of custom caching

What's Changed

Full Changelog: master...2.1.0

@wesleytodd
Copy link
Member Author

For some reason it is not letting me add @Phillip9587 as a reviewer, so here is a ping.

Copy link
Member

@UlisesGascon UlisesGascon left a comment

Choose a reason for hiding this comment

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

LGTM!

Copy link
Member

@UlisesGascon UlisesGascon left a comment

Choose a reason for hiding this comment

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

Actually.. we need to change the HISTORY.md to include the release date, etc.

@wesleytodd
Copy link
Member Author

Fixed a second error I made while resolving the conflict in package.json. We had one commit that bumped all the ranges from ~ to ^ and also one to rev type-is and one to remove destroy. I took the wrong side of that merge conflict, both should be resolved now.

@wesleytodd
Copy link
Member Author

Oh, and I will update the history date if folks have reviewed the change set (thanks or the catch on the destroy dep @bjohansebas!) when we release.

@Phillip9587
Copy link
Contributor

LGTM

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

Successfully merging this pull request may close these issues.

5 participants