Skip to content

Merge new-build-process #243

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

Open
wants to merge 7 commits into
base: no-sidebar
Choose a base branch
from

Conversation

chris-steele
Copy link
Contributor

@chris-steele chris-steele commented May 5, 2023

#231, #233

Update

  • use rollup as the bundler rather than requirejs
  • be able to fully utilise ES6 and beyond
  • be backwards compatible with existing code base
  • include React support
  • compile sources in situ rather than copying them
  • watch sources and compile on change
  • no longer need to restart backend stack when working on UI
  • cache bundler output for faster builds

Much of this work has been inspired by the Adapt framework.

Instructions

  • checkout no-sidebar-new-build-process
  • run npm start
  • wait for UI successfully built notice
  • edit js/jsx/hbs*/less and observe console
  • reload browser to see changes

*core index.hbs/loading.hbs are not pre-compiled and will be cached by the express templating engine; so in the (unlikely) event these are edited the process may need to be restarted

Notes

@chris-steele chris-steele marked this pull request as draft May 22, 2023 12:08
@chris-steele chris-steele reopened this May 22, 2023
@chris-steele chris-steele marked this pull request as ready for review May 22, 2023 12:29
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
Status: Needs Reviewing
Development

Successfully merging this pull request may close these issues.

1 participant