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

chore: remove nodemon #16

Open
wants to merge 2 commits into
base: master
Choose a base branch
from
Open

chore: remove nodemon #16

wants to merge 2 commits into from

Conversation

thgh
Copy link

@thgh thgh commented Jan 26, 2023

Hi, this is just a suggestion. Based on my benchmarks, this improves startup time in this repo from 2220 ms to 1975 ms. In my personal project it improves startup time from 1900 ms to 1391 ms. (Best of 10 runs)

@thgh thgh changed the title chore: replace ts-node with swc-node chore: remove nodemon Feb 2, 2023
@thgh
Copy link
Author

thgh commented Feb 2, 2023

omg, I have been using nodemon for months based on the demo repo, turns out there is no reason to restart the app for every change. This PR delivers the intended Payload CMS behaviour.

Edit: it's for reloading server files

@jacobsfletch
Copy link
Member

Related: #20

@thgh
Copy link
Author

thgh commented Jul 11, 2023

Related: payloadcms/payload#2563

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.

2 participants