Skip to content

Do not include databases with version 0 in databases() results - #448

Merged
inexorabletash merged 1 commit into
mainfrom
databases-no-zeros
Apr 24, 2025
Merged

Do not include databases with version 0 in databases() results#448
inexorabletash merged 1 commit into
mainfrom
databases-no-zeros

Conversation

@inexorabletash

@inexorabletash inexorabletash commented Apr 11, 2025

Copy link
Copy Markdown
Member

When a database is first created it is treated as an upgrade from version 0 to the new version (the default is 1). Per WPT, all implementations exclude these in-progress databases from the enumeration returned by databases(), so reflect this in the algorithm steps.

Closes #442


Preview | Diff

When a database is first created it is treated as an upgrade from
version 0 to the new version (the default is 1). Per WPT, all
implementations exclude these in-progress databases from the
enumeration returned by databases(), so reflect this in the algorithm
steps.
@inexorabletash

Copy link
Copy Markdown
Member Author

@stelar7 please take a look?

@inexorabletash
inexorabletash merged commit ee73e87 into main Apr 24, 2025
@inexorabletash
inexorabletash deleted the databases-no-zeros branch April 24, 2025 21:48
github-actions Bot added a commit that referenced this pull request Apr 24, 2025
SHA: ee73e87
Reason: push, by inexorabletash

Co-authored-by: github-actions[bot] <41898282+github-actions[bot]@users.noreply.github.com>
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.

Possible wrong WPT test?

3 participants