Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
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
Switch from
webpack
toesbuild
#19Switch from
webpack
toesbuild
#19Changes from 28 commits
37dbc4e
9776074
aa114b6
b60c024
9c49f0f
e6c2536
6724342
d09f16d
6284bb3
1645f5c
f79ea92
a8605aa
905fe30
b78c91f
e4b6af2
17bd7d5
a13cb07
9ff7335
215425a
b27ba4c
9e3debe
6121837
92874ad
f638cb2
4d6625e
2808462
19caa5a
8162be9
3037b6b
1138071
d50b38e
592039e
2a1e0f8
0a8b5ea
13dd9da
File filter
Filter by extension
Conversations
Jump to
There are no files selected for viewing
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I think
-port
has two dashes:(also why do we need to pass it if it's the same value as the default?)
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
we don't need to pass it, you are right. Removed
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I'd mention here
esbuild
and the@craftamap/esbuild-plugin-html
plugin.There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
added, but maybe it's better to vendor it?
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I wonder if we should mention here that if users use their own template, they have to add the
new EventSource('/esbuild')...
script manually if they want to keep livereload working.There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
These are needed because otherwise it would use the default
[dir]/[name]
right? Not sure if worth a comment.There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I actually do not no, they were here before me. Will check
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Should we add a minimal graceful shutdown for esbuild server and node http server?
We'll listen for
SIGINT
andSIGTERM
signals.For esbuild:
https://esbuild.github.io/api/#build
For node HTTP server:
https://nodejs.org/api/http.html#serverclosecallback
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Done: 592039e
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.