-
Notifications
You must be signed in to change notification settings - Fork 1.6k
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
Supporting cordova-cli@9 or cordova-lib@9 #1034
Open
sagrawal31
wants to merge
73
commits into
arnesson:master
Choose a base branch
from
wizpanda:master
base: master
Could not load branches
Branch not found: {{ refName }}
Loading
Could not load tags
Nothing to show
Loading
Are you sure you want to change the base?
Some commits from the old base branch may be removed from the timeline,
and old review comments may become outdated.
Open
Changes from 5 commits
Commits
Show all changes
73 commits
Select commit
Hold shift + click to select a range
2e1fae3
Supporting cordova-cli@9 or cordova-lib@9
sagrawal31 08f9b17
Trying to fix the build
sagrawal31 609211e
Removing old cordova-android because of a failing test case
sagrawal31 a2c57ad
Only installing xcode for ios platform
sagrawal31 247b6a2
Fixing old failing Android tests by not installing the cordova & plat…
sagrawal31 46a7bd1
Fixes issues cause by Firebase SDK updates on 5 April 2019 (https://f…
dpa99c eee2cfe
Remove obsolete <service> entry for FirebasePluginInstanceIDService.
dpa99c 17eb7c4
Remove unnecessary extra <config-file> block which can lead to race c…
dpa99c 289706f
Remove redundant build-extras.gradle
dpa99c ad537b6
Merge pull request #2 from wizpanda/firebase-changes
sagrawal31 878a4c5
3.0.0
sagrawal31 2333790
Updating README.md
sagrawal31 385ce85
Adding npm version badge.
sagrawal31 ffec9e3
Added ability to log js stack trace on android.
b674176
Missed adding file to plugin.xml
42c66b0
Fixed error if stacktrace is missing data and js bug if stackTrace wa…
c1d4d10
Fixed ios error logging and added ability to log stack traces.
bd921fa
Add createChannel function for Android 0
maximilianloy 3046ef9
Merge pull request #9 from maximilianloy/android-createchannel
sagrawal31 1911802
Formatted the code
sagrawal31 b8eb565
Only changing the indentation to 4 spaces.
sagrawal31 06de856
3.1.0
sagrawal31 b8ec09f
Trying to fix the Travis CI build.
sagrawal31 809d7f5
Merge branch 'master' into master
distinctdan b50e88d
Merge pull request #8 from distinctdan/master
sagrawal31 bfd4bf6
Some code cleanup and changes to log stacktrace in Firebase.
sagrawal31 428f306
3.2.0
sagrawal31 680ac02
Backward compatibility fixes to support breaking changes released by …
sagrawal31 32f40c7
Merge pull request #14 from wizpanda/firebase-breaking-backward
sagrawal31 cd3abfd
3.3.0
sagrawal31 92af2ff
First version of changes to make the app build after breaking changes…
sagrawal31 f72326d
Changing the minimum required cordova-android version to 8.0.0
sagrawal31 0379213
Updating README.md
sagrawal31 bc4cf4e
Merge pull request #13 from wizpanda/google-breaking-changes
sagrawal31 ed019d1
GitHub issue template cleanup. Updating JS files to use "const" or "l…
sagrawal31 066042f
4.0.0
sagrawal31 65e6925
Formatting plugin.xml to use 4 spaces.
sagrawal31 5219529
Fixed typo in firebase-browser.js fixes #16
sagrawal31 9d968eb
Only formatting the JS files.
sagrawal31 688b634
4.0.1
sagrawal31 6a965f6
Breaking Change: Merging `logJSError` to `logError` method for consis…
sagrawal31 03ee4ff
Updating the API doc.
sagrawal31 d72c302
Generated "Table of Contents" via markdown-toc generator.
sagrawal31 a2a1377
Handling a few edge cases for "logError" method.
sagrawal31 70b2c6c
Updating README.
sagrawal31 999f666
Merge pull request #21 from wizpanda/android-fixes
sagrawal31 e4d4dc0
Merge pull request #1 from wizpanda/master
distinctdan 263507e
Added more docs for passing a stacktrace to logError
distinctdan 0d798b5
Merge pull request #22 from distinctdan/master
sagrawal31 e60cd91
4.1.0
sagrawal31 b175b42
Updating CHANGELOG & docs.
sagrawal31 818f0cf
Breaking Change: Moving to CocoaPods to manage Firebase, Crashlytics …
sagrawal31 0d4719d
Using optimistic operator to specify Firebase dependencies for iOS.
sagrawal31 1e8a384
Updating README.
sagrawal31 b86af91
More clear documentation about different cordova platform versions.
sagrawal31 08d71bd
Merge pull request #23 from wizpanda/ios-pods
sagrawal31 dcfd859
5.0.0
sagrawal31 ebf3191
Updating README.
sagrawal31 72ee4fc
Fix for ampersand in the app name
teknepal f8bfb82
Reducing the Firebase iOS dependencies to 5.x to avoid breaking changes.
sagrawal31 310bbf7
Removed deprecated method "getInstanceId"
sagrawal31 fe059bf
Merge pull request #28 from wizpanda/ios-fixes
sagrawal31 1a0f3e7
5.1.0
sagrawal31 dc7f9ad
changed const to var
teknepal d4fa7f6
Merge pull request #26 from teknepal/fix_ampersand_app_name
sagrawal31 c1539e4
Improving the JS code. Using chalk to highlights a few logs.
sagrawal31 487f3be
Some docs improvement.
sagrawal31 5b1a526
Merge pull request #30 from wizpanda/js-improvement
sagrawal31 d1c5774
Some more logging improvement.
sagrawal31 19828d1
5.1.1
sagrawal31 645d5f3
Using tabular format for different version of plugin.
sagrawal31 eb15341
Moving the difference doc on the top & some cleanup.
sagrawal31 154596a
Updating README for diverting developers to new repository.
sagrawal31 File filter
Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
There are no files selected for viewing
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Original file line number | Diff line number | Diff line change |
---|---|---|
|
@@ -32,7 +32,7 @@ | |
"version": "node ./bump-version.js && git add -A plugin.xml", | ||
"postversion": "git push && git push --tags && npm publish --verbose", | ||
"test": "npm run test:ios && npm run test:android && npm run test:browser", | ||
"test:android": "npm run test:cordova@6.5.0:android@6.4.0 && npm run test:cordova@7.1.0:[email protected] && npm run test:cordova@8.0.0:android@7.1.0", | ||
"test:android": "npm run test:cordova@7.1.0:android@7.1.0 && npm run test:cordova@8.0.0:[email protected] && npm run test:cordova@9.0.0:android@8.0.0", | ||
"test:browser": "npm run test:[email protected]:[email protected] && npm run test:[email protected]:[email protected] && npm run test:[email protected]:[email protected]", | ||
"test:ios": "npm run test:[email protected]:[email protected] && npm run test:[email protected]:[email protected] && npm run test:[email protected]:[email protected]", | ||
"test:[email protected]:[email protected]": "bash ./test/test-default.sh 6.5.0 ios 4.5.4", | ||
|
@@ -45,5 +45,7 @@ | |
"test:[email protected]:[email protected]": "bash ./test/test-default.sh 7.1.0 browser 5.0.3", | ||
"test:[email protected]:[email protected]": "bash ./test/test-default.sh 8.0.0 browser 5.0.3" | ||
}, | ||
"dependencies": {} | ||
"dependencies": { | ||
"xcode": "^2.0.0" | ||
} | ||
} |
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
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.
Is this what made the build pass? If yes, I'll submit a separate PR with this fix, so all the broken PRs would start passing after rebasing.
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.
Not really, all but yes, it did fixed a few test cases. Since the cordova & platform version passed to this script from the parent script are same hence there was no point reinstalling it again so I commented out that. But I would love to see the detailed error when this script was trying to install the same cordova & platform.
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.
I would also love to help you on rebasing PRs & getting them merged which are good to go because I saw various good PRs
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.
how can we go forward ?
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.
@felipeclopes can tell anything
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.
I'm not really sure hot to proceed. It seems that the first step would be to try to fix the test cases that are broken. If you look at the list of PRs to merge, the only ones passing are the ones who removed the tests for older Cordova versions and this one.
Maybe a separate PR just with this fix of the build cold help the other PRs to rebase and get a green status on the build system.
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.
In my thinking, we can start with merging this branch to master then start rebasing other PR with master so most of the PRs will by default pass
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.
@felipeclopes any update? I'll still suggest to move forward with this