erisu opened a new pull request, #149:
URL: https://github.com/apache/cordova-browser/pull/149
### Motivation and Context
<!-- Why is this change required? What problem does it solve? -->
<!-- If it fixes an open issue, please link to the issue here. -->
- Build `cordova.js` from source using npm's `prepare` hook.
- Do not bundle unnecessary files and directories.
Closes #99
### Description
<!-- Describe your changes in detail -->
- Build `cordova.js` from source using npm's `prepare` hook
- Remove and ignore previously prebuilt `cordova.js` files
- Stop bundling `cordova-js-src` (Changes from #99)
- Clean up `npm pack` output by excluding:
- dotfiles
- test directories, related configuration files, and output artifacts
- log files and directories
- previous `npm pack` output
### Testing
<!-- Please describe in detail how you tested your changes. -->
- Successfully built tarball using `npm pack`
- Successfully ran `npm cit` from clean repo
- The generated `cordova.js` is slightly different because its appears the
file in repo is using main branch of Cordova-JS and not the official release of
`cordova-js`.
- Tested `cordova platform add` with the tarball from `npm pack`
--
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.
To unsubscribe, e-mail: [email protected]
For queries about this service, please contact Infrastructure at:
[email protected]
---------------------------------------------------------------------
To unsubscribe, e-mail: [email protected]
For additional commands, e-mail: [email protected]
