Skip to content

Commit 79035c0

Browse files
committed
resolve todos
1 parent 5e19446 commit 79035c0

File tree

6 files changed

+4
-77
lines changed

6 files changed

+4
-77
lines changed

.github/workflows/update-api-reports.yml

Lines changed: 2 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -21,7 +21,8 @@ jobs:
2121
- name: Yarn install
2222
run: yarn
2323
- name: Update API reports
24-
run: yarn ts-node-script scripts/exp/update-api-reports.ts
24+
# API reports are generated as part of the build
25+
run: yarn build
2526
id: update-api-reports
2627
- name: Commit & Push changes
2728
uses: EndBug/add-and-commit@v7

package.json

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -22,7 +22,7 @@
2222
],
2323
"scripts": {
2424
"dev": "lerna run --parallel --scope @firebase/* --scope firebase dev",
25-
"build": "lerna run --scope @firebase/* --scope firebase --ignore @firebase/rules-unit-testing --ignore @firebase/firestore --ignore build",
25+
"build": "lerna run --scope @firebase/* --scope firebase --ignore @firebase/rules-unit-testing --ignore @firebase/firestore build",
2626
"build:changed": "ts-node-script scripts/ci-test/build_changed.ts",
2727
"release:prepare": "lerna run --scope @firebase/* add-compat-overloads && lerna run --scope @firebase/* typings:public",
2828
"link:packages": "lerna exec --scope @firebase/* --scope firebase -- yarn link",

packages/performance-compat/src/index.ts

Lines changed: 0 additions & 7 deletions
Original file line numberDiff line numberDiff line change
@@ -25,13 +25,6 @@ import { PerformanceCompatImpl } from './performance';
2525
import { name as packageName, version } from '../package.json';
2626
import { FirebasePerformance as FirebasePerformanceCompat } from '@firebase/performance-types';
2727

28-
// TODO: move it to the future performance-compat-types package
29-
declare module '@firebase/component' {
30-
interface NameServiceMapping {
31-
'performance-compat': FirebasePerformanceCompat;
32-
}
33-
}
34-
3528
function registerPerformanceCompat(firebaseInstance: _FirebaseNamespace): void {
3629
firebaseInstance.INTERNAL.registerComponent(
3730
new Component(

packages/performance-types/index.d.ts

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -114,6 +114,6 @@ export interface PerformanceTrace {
114114

115115
declare module '@firebase/component' {
116116
interface NameServiceMapping {
117-
'performance': FirebasePerformance;
117+
'performance-compat': FirebasePerformance;
118118
}
119119
}

packages/remote-config-compat/src/index.ts

Lines changed: 0 additions & 7 deletions
Original file line numberDiff line numberDiff line change
@@ -26,13 +26,6 @@ import { RemoteConfigCompatImpl } from './remoteConfig';
2626
import { name as packageName, version } from '../package.json';
2727
import { RemoteConfig as RemoteConfigCompat } from '@firebase/remote-config-types';
2828

29-
// TODO: move it to remote-config-types package
30-
declare module '@firebase/component' {
31-
interface NameServiceMapping {
32-
'remoteConfig-compat': RemoteConfigCompat;
33-
}
34-
}
35-
3629
function registerRemoteConfigCompat(
3730
firebaseInstance: _FirebaseNamespace
3831
): void {

scripts/exp/update-api-reports.ts

Lines changed: 0 additions & 60 deletions
This file was deleted.

0 commit comments

Comments
 (0)