Skip to content

Development Tooling (Rollup + BrowserSync)#7

Closed
lukeed wants to merge 6 commits intomasterfrom
build
Closed

Development Tooling (Rollup + BrowserSync)#7
lukeed wants to merge 6 commits intomasterfrom
build

Conversation

@lukeed
Copy link
Collaborator

@lukeed lukeed commented Dec 7, 2016

Using Rollup, there is (currently) no way to both recompile changes and reload the current server. Forget about HMR! The Rollup dev process needs a third party to breach the gap & coordinate tasks.

This PR brings back the webpack-dev-server utility/behavior while still using Rollup.

Note: This does not alter the bundle's contents or size in any way.

Changes:

  • Replaced http-server with browser-sync
  • Renamed & relocatd todomvc-* static assets.
  • Installed Fly for lightweight task watcher & manager
  • (Greatly) Simplified all npm scripts
  • Removed all unused dependencies (Webpack & co)

I could have used fly-uglify and re-applied the UglifyJS2 config using its JS API, but the previous CLI string was already done & straightforward, so I just passed it thru fly-shell instead.

@lukeed
Copy link
Collaborator Author

lukeed commented Dec 7, 2016

Unless surge or gh-pages requires the http-server because it's simple / static?

@developit
Copy link
Owner

surge and gh-pages just use the contents of build, I was just using http-server to test. This still outputs to build/ for production builds right (npm run build)?

@lukeed
Copy link
Collaborator Author

lukeed commented Dec 7, 2016

That's what I thought. Yup!

@lukeed
Copy link
Collaborator Author

lukeed commented Dec 8, 2016

@developit This is your call (obviously 😜). Won't care if you trash it, but if you're waiting for me to merge it, I can't.

@developit
Copy link
Owner

Going to check it out locally so I can get acquainted with the dev workflow and check a couple things (sourcemaps etc)

@developit
Copy link
Owner

(sorry for taking an eternity on this, toiling over 7.2 release and some other stuff!)

@lukeed
Copy link
Collaborator Author

lukeed commented Dec 18, 2016

Cool! No matter to me; can go either way.

@lukeed
Copy link
Collaborator Author

lukeed commented Mar 11, 2019

Wow this is old 😂

@lukeed lukeed closed this Mar 11, 2019
@lukeed lukeed deleted the build branch March 11, 2019 17:25
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