Skip to content

Conversation

@laktek
Copy link
Contributor

@laktek laktek commented Apr 15, 2024

What kind of change does this PR introduce?

Updates functions-js dependency to 2.3.0

@laktek laktek requested a review from soedirgo April 15, 2024 01:30
@laktek laktek force-pushed the bump-functions-js-230 branch from 7aa4e83 to ec840c6 Compare April 15, 2024 01:33
@coveralls
Copy link

Pull Request Test Coverage Report for Build 8682040794

Details

  • 0 of 0 changed or added relevant lines in 0 files are covered.
  • No unchanged relevant lines lost coverage.
  • Overall coverage remained the same at 65.714%

Totals Coverage Status
Change from base Build 8655762334: 0.0%
Covered Lines: 90
Relevant Lines: 117

💛 - Coveralls

@laktek laktek merged commit f22a410 into master Apr 15, 2024
@laktek laktek deleted the bump-functions-js-230 branch April 15, 2024 01:56
mandarini pushed a commit that referenced this pull request Oct 2, 2025
## What kind of change does this PR introduce?

Bug fix

## What is the current behavior?

Type checking and/or linting does not pass for some configurations, per
supabase/auth-js#967 and
supabase/auth-js#1017

## What is the new behavior?

We're now asserting that `parts[1]` is a string (aka isn't undefined),
since we know from a previous check that the array length is 3.

## Additional context

Fixes #967 and fixes #1017

Co-authored-by: Kang Ming <[email protected]>
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.

4 participants