Skip to content

test: enable posix path tests#1658

Merged
jsignell merged 1 commit intostac-utils:v2from
ircwaves:v2-posix-path
Mar 16, 2026
Merged

test: enable posix path tests#1658
jsignell merged 1 commit intostac-utils:v2from
ircwaves:v2-posix-path

Conversation

@ircwaves
Copy link
Copy Markdown
Member

a trivial one to get started on the #1657 effort

@ircwaves ircwaves changed the base branch from main to v2 March 16, 2026 14:55
@ircwaves ircwaves changed the title V2: enable posix path tests chore: enable posix path tests Mar 16, 2026
@ircwaves ircwaves changed the title chore: enable posix path tests test: enable posix path tests Mar 16, 2026
Copy link
Copy Markdown
Member

@jsignell jsignell left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

nice one! I guess in this case we can delay the decision about: "Is link.get_target_str a method that we need to deprecate? Or is it weird and unused and we can just delete it in which case we can xfail tests for it."

@ircwaves
Copy link
Copy Markdown
Member Author

Is link.get_target_str a method that we need to deprecate? Or is it weird and unused and we can just delete it in which case we can xfail tests for it.

That does feel like that falls more into the test_link.py test update. But I think deprecation is probably appropriate, and the less-disruptive road we're navigating, right? I'll pick up that update soon and include the deprecation.

@jsignell jsignell merged commit 7590b71 into stac-utils:v2 Mar 16, 2026
3 of 5 checks passed
@jsignell jsignell mentioned this pull request Mar 19, 2026
23 tasks
@jsignell jsignell mentioned this pull request Mar 30, 2026
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.

2 participants