Skip to content
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

build: build cordova.js during npm prepare #1197

Merged
merged 3 commits into from
Nov 16, 2021

Conversation

erisu
Copy link
Member

@erisu erisu commented Nov 16, 2021

Motivation and Context

This will build cordova.js from source using npm's prepare hook.

Description

  • build cordova.js from source using npm's prepare hook
  • copy prepared cordova.js before running Xcode tests
  • delete and ignore the prebuilt cordova.js files we had
  • do not bundle cordova-js-src anymore, since support for browserify builds has been discontinued for a while now

Testing

  • npm t
  • cordova platform add
  • cordova build ios
  • cordova run ios

@erisu erisu added this to the 7.0.0 milestone Nov 16, 2021
@codecov-commenter
Copy link

codecov-commenter commented Nov 16, 2021

Codecov Report

Merging #1197 (7cfc4f7) into master (b1b2141) will not change coverage.
The diff coverage is n/a.

Impacted file tree graph

@@           Coverage Diff           @@
##           master    #1197   +/-   ##
=======================================
  Coverage   75.15%   75.15%           
=======================================
  Files          13       13           
  Lines        1658     1658           
=======================================
  Hits         1246     1246           
  Misses        412      412           

Continue to review full report at Codecov.

Legend - Click here to learn more
Δ = absolute <relative> (impact), ø = not affected, ? = missing data
Powered by Codecov. Last update b1b2141...7cfc4f7. Read the comment docs.

@erisu erisu marked this pull request as ready for review November 16, 2021 12:54
Copy link
Contributor

@raphinesse raphinesse left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

I cannot test this, but it looks good to me (my two comments aside)

bin/lib/create.js Outdated Show resolved Hide resolved
package.json Outdated Show resolved Hide resolved
@erisu erisu merged commit 72a09a8 into apache:master Nov 16, 2021
@erisu erisu deleted the feat/build-cordova-js branch November 16, 2021 15:26
gazben pushed a commit to apicore-engineering/cordova-ios that referenced this pull request Aug 26, 2022
* build: build cordova.js during npm prepare w/ refactor
* chore: apply copySync suggestion
* dep: move cordova-js as devDep

Co-authored-by: Raphael von der Grün <[email protected]>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants