-
Notifications
You must be signed in to change notification settings - Fork 90
Build with Vite #1045
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
base: dev
Are you sure you want to change the base?
Build with Vite #1045
Conversation
| if (graphHopperResponseFile) { | ||
| nock('https://graphhopper.com') | ||
| // Turn on log in order to view nock request URLs. | ||
| .log(console.log) |
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.
nock.log no longer exists in recent node versions.
| "mastarm": "^5.3.1", | ||
| "md5-file": "^4.0.0", | ||
| "nock": "^9.0.14", | ||
| "nock": "^13.5.6", |
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.
Nock 14+ require adding other mocks that are currently missing. Lmk is that's worth it in this PR.
This reverts commit eea1536.
|
Build passes!! 🎉🎉🎉 |
Checklist
devbefore they can be merged tomaster)Description
This PR replaces mastarm with Vite as the build system (for dev and production). Other parts involving mastarm remain unchanged.
mastarm buildormastarm deployUseful commands:
env YAML_CONFIG=<path-to-frontend>/env.yml yarn startenv YAML_CONFIG=<path-to-frontend>/env.yml yarn build