feat: update storage extension to v2.0.0

[tylanderson]

Description:

StorageScheme platform property allows templated uri's, so arbitrary additional properties are allowed to be set on StorageScheme objects

Includes best-effort a migration from v1 to v2, with currently defined best-practices

PR Checklist:

• Pre-commit hooks pass (run pre-commit run --all-files)
• Tests pass (run pytest)
• Documentation has been updated to reflect changes, if applicable
• This PR maintains or improves overall codebase code coverage.
• Changes are added to the CHANGELOG. See the docs for information about adding to the changelog.

[codecov]

Codecov Report

❌ Patch coverage is 86.41115% with 39 lines in your changes missing coverage. Please review.
✅ Project coverage is 92.03%. Comparing base (d4eecb9) to head (6a4f054).

Files with missing lines	Patch %	Lines
pystac/extensions/storage.py	85.97%	25 Missing and 14 partials ⚠️

Additional details and impacted files

@@            Coverage Diff             @@
##             main    #1561      +/-   ##
==========================================
- Coverage   92.22%   92.03%   -0.20%     
==========================================
  Files          55       55              
  Lines        8414     8622     +208     
  Branches      973     1004      +31     
==========================================
+ Hits         7760     7935     +175     
- Misses        466      486      +20     
- Partials      188      201      +13     

☔ View full report in Codecov by Sentry.
📢 Have feedback on the report? Share it here.

🚀 New features to boost your workflow:

• ❄️ Test Analytics: Detect flaky tests, report on failures, and find test suite problems.

[gadomski]

I think we have unnecessary new extensions here ... both schemes and refs can just be classes, not extensions themselves?

[thomas-maschler]

Can you also add GCS = "gcp-gs"

[thomas-maschler]

generally, having parity to the v1 Enum would be great

[tylanderson]

I didn't see best practice GCP storage defined here, so did not add as a possible enum here, but as written schemes can be set to any string. I think opening a PR in the extension w/ a suggested schema to use for GCP storage would make sense before implemented here.

[gadomski]

Maybe I missed it, but we should test the migration path from v1 to v2.

[gadomski]

Looks like the docs need to be reverted back to a single class?

[pjonsson]

Missing closing brace.

[tylanderson]

Maybe I missed it, but we should test the migration path from v1 to v2.

I'm taking a look at what adding a migration hook would require, but I think there will be some cases that will be harder to migrate cleanly. (e.g. azure where we need account id, or the platforms that exist in v1 but not v2 e.g. IBM, alibaba)

[jsignell]

Maybe I missed it, but we should test the migration path from v1 to v2.

I'm taking a look at what adding a migration hook would require, but I think there will be some cases that will be harder to migrate cleanly. (e.g. azure where we need account id, or the platforms that exist in v1 but not v2 e.g. IBM, alibaba)

I think raising a warning in those cases and then leaving the data unchanged will be sufficient.

[gadomski]

Sorry about the long delay, thanks for the contribution!