Skip to content
This commit does not belong to any branch on this repository, and may belong to a fork outside of the repository.

Commit a2c1c24

Browse files
committedNov 29, 2022
Merge branch 'master' into mila/BloomFilter
2 parents d6c5756 + 09dfc3a commit a2c1c24

File tree

7 files changed

+24
-8
lines changed

7 files changed

+24
-8
lines changed
 

‎.changeset/six-brooms-listen.md

Lines changed: 5 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -0,0 +1,5 @@
1+
---
2+
'@firebase/auth': patch
3+
---
4+
5+
move selenium-webdriver to devDependencies

‎.github/workflows/test-changed-auth.yml

Lines changed: 4 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -38,7 +38,10 @@ jobs:
3838
run: xvfb-run yarn test:changed auth
3939
test-firefox:
4040
name: Test Auth on Firefox If Changed
41-
runs-on: ubuntu-latest
41+
# Whatever version of Firefox comes with 22.04 is causing Firefox
42+
# startup to hang when launched by karma. Need to look further into
43+
# why.
44+
runs-on: ubuntu-20.04
4245

4346
steps:
4447
- name: install Firefox stable

‎.github/workflows/test-changed-firestore.yml

Lines changed: 4 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -34,7 +34,10 @@ jobs:
3434

3535
test-firefox:
3636
name: Test Firestore on Firefox If Changed
37-
runs-on: ubuntu-latest
37+
# Whatever version of Firefox comes with 22.04 is causing Firefox
38+
# startup to hang when launched by karma. Need to look further into
39+
# why.
40+
runs-on: ubuntu-20.04
3841

3942
steps:
4043
- name: install Firefox stable

‎.github/workflows/test-changed.yml

Lines changed: 4 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -34,7 +34,10 @@ jobs:
3434

3535
test-firefox:
3636
name: Test Packages With Changed Files in Firefox
37-
runs-on: ubuntu-latest
37+
# Whatever version of Firefox comes with 22.04 is causing Firefox
38+
# startup to hang when launched by karma. Need to look further into
39+
# why.
40+
runs-on: ubuntu-20.04
3841

3942
steps:
4043
- name: Checkout Repo

‎README.md

Lines changed: 5 additions & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -190,14 +190,16 @@ scope](https://www.npmjs.com/search?q=scope%3Afirebase) on NPM.
190190

191191
### Testing the SDK Locally
192192

193-
Please be sure to build your repo before proceeding any further.
193+
Please be sure your product's package has been built before proceeding any further. (If you haven't built this repo before, make sure to run `yarn build` at the root)
194194
In order to manually test your SDK changes locally, you must use [yarn link](https://classic.yarnpkg.com/en/docs/cli/link):
195195

196196
```shell
197197
$ cd packages/firebase
198198
$ yarn link # initialize the linking to the other folder
199-
$ cd ../<my-test-app-dir> # cd into your personal project directory
200-
$ yarn link firebase # tell yarn to use the locally built firebase SDK instead
199+
$ cd ../packages/<my-product> # Example: $ cd packages/database
200+
$ yarn link # link your product to make it available elsewhere
201+
$ cd <my-test-app-dir> # cd into your personal project directory
202+
$ yarn link firebase @firebase/<my-product> # tell yarn to use the locally built firebase SDK instead
201203
```
202204

203205
This will create a symlink and point your `<my-test-app-dir>` to the locally built version of the firebase SDK.

‎packages/auth-compat/package.json

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -51,7 +51,6 @@
5151
"@firebase/component": "0.5.21",
5252
"@firebase/util": "1.7.3",
5353
"node-fetch": "2.6.7",
54-
"selenium-webdriver": "4.5.0",
5554
"tslib": "^2.1.0"
5655
},
5756
"license": "Apache-2.0",
@@ -61,6 +60,7 @@
6160
"rollup": "2.79.1",
6261
"rollup-plugin-replace": "2.2.0",
6362
"rollup-plugin-typescript2": "0.31.2",
63+
"selenium-webdriver": "4.5.0",
6464
"typescript": "4.2.2"
6565
},
6666
"repository": {

‎packages/auth/package.json

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -106,7 +106,6 @@
106106
"@firebase/logger": "0.3.4",
107107
"@firebase/util": "1.7.3",
108108
"node-fetch": "2.6.7",
109-
"selenium-webdriver": "4.5.0",
110109
"tslib": "^2.1.0"
111110
},
112111
"license": "Apache-2.0",
@@ -118,6 +117,7 @@
118117
"rollup": "2.79.1",
119118
"rollup-plugin-sourcemaps": "0.6.3",
120119
"rollup-plugin-typescript2": "0.31.2",
120+
"selenium-webdriver": "4.5.0",
121121
"typescript": "4.2.2",
122122
"@types/express": "4.17.14"
123123
},

0 commit comments

Comments
 (0)
Please sign in to comment.