Skip to content

fix: firebase_core folder order by parsing the version - #410

Merged
russellwheatley merged 3 commits into
invertase:mainfrom
frederikstonge:features/fred/fix-version-order
Jul 23, 2025
Merged

russellwheatley merged 3 commits into
invertase:mainfrom
frederikstonge:features/fred/fix-version-order

Conversation

@frederikstonge

Copy link
Copy Markdown
Contributor

Description

There was an issue where folder ordering by name would think firebase_core-3.3.0 was higher than firebase_core-3.14.0. I used pub_semver to parse the version to make sure we get the latest version. It might look redundant, but I decided to filter out other package folders to make sure we don't parse the version of all the pub cache folders.

Type of Change

  • ✨ feat -- New feature (non-breaking change which adds functionality)
  • 🛠️ fix -- Bug fix (non-breaking change which fixes an issue)
  • ❌ ! -- Breaking change (fix or feature that would cause existing functionality to change)
  • 🧹 refactor -- Code refactor
  • ✅ ci -- Build configuration change
  • 📝 docs -- Documentation
  • 🗑️ chore -- Chore
@frederikstonge frederikstonge changed the title Fix firebase_core folder order by parsing the version Jun 25, 2025
@frederikstonge

Copy link
Copy Markdown
Contributor Author

@russellwheatley Could you review this please?

@github-actions github-actions Bot added the Needs Attention OP created or responded to issue and it needs attention. label Jul 22, 2025
@russellwheatley
russellwheatley merged commit 2cfefc3 into invertase:main Jul 23, 2025

This branch was previously deployed

1 inactive deployment
CI Environment — fb08e9cc Deployed Jul 23, 2025 by frederikstonge
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Needs Attention OP created or responded to issue and it needs attention.

2 participants