Skip to content

Handle empty Remote Config parameter groups - #1133

Merged
jeromegamez merged 1 commit into
8.xfrom
fix/1132-empty-parameter-groups
Sep 4, 2026
Merged

jeromegamez merged 1 commit into
8.xfrom
fix/1132-empty-parameter-groups

Conversation

@jeromegamez

@jeromegamez jeromegamez commented Sep 4, 2026

Copy link
Copy Markdown
Member

Fixes #1132

@codecov

codecov Bot commented Sep 4, 2026

Copy link
Copy Markdown

Codecov Report

✅ All modified and coverable lines are covered by tests.
✅ All tests successful. No failed tests found.

❗ There is a different number of reports uploaded between BASE (49d099f) and HEAD (5ee5009). Click for more details.

HEAD has 3 uploads less than BASE
Flag BASE (49d099f) HEAD (5ee5009)
grpc 1 0
emulator 1 0
integration 1 0

Impacted file tree graph

@@              Coverage Diff              @@
##                8.x    #1133       +/-   ##
=============================================
- Coverage     87.83%   41.08%   -46.75%     
  Complexity     1497     1497               
=============================================
  Files           147      147               
  Lines          4199     4184       -15     
=============================================
- Hits           3688     1719     -1969     
- Misses          511     2465     +1954     

@jeromegamez
jeromegamez force-pushed the fix/1132-empty-parameter-groups branch from b428e8a to 5ee5009 Compare September 4, 2026 22:15
@jeromegamez
jeromegamez merged commit 4fce0bc into 8.x Sep 4, 2026
24 checks passed
@jeromegamez
jeromegamez deleted the fix/1132-empty-parameter-groups branch September 4, 2026 22:17
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

FirebaseConfig method get() will crash if one of parameterGroups return an empty array

1 participant