Skip to content

Fix fields argument to CLI#797

Merged
jsignell merged 5 commits intomainfrom
js/fields-cli
May 12, 2025
Merged

Fix fields argument to CLI#797
jsignell merged 5 commits intomainfrom
js/fields-cli

Conversation

@jsignell
Copy link
Copy Markdown
Member

@jsignell jsignell commented May 9, 2025

Related Issue(s):

Description:

This is not the most beautiful PR, but this now works:

stac-client search https://earth-search.aws.element84.com/v1 -c sentinel-2-l2a --bbox -72.5 40.5 -72 41 --datetime 2020-01-01/2020-01-11 --fields=-geometry,-assets --max-items 1

PR Checklist:

  • Code is formatted
  • Tests pass
  • Changes are added to CHANGELOG.md

@jsignell jsignell requested a review from gadomski May 9, 2025 18:59
Comment thread tests/test_cli.py
@jsignell jsignell requested a review from gadomski May 9, 2025 19:34
Comment thread tests/test_cli.py
@jsignell jsignell requested a review from gadomski May 9, 2025 20:44
@jsignell jsignell merged commit 5fae268 into main May 12, 2025
14 checks passed
@jsignell jsignell deleted the js/fields-cli branch May 12, 2025 13:35
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.

Unable to define multiple exclusions on --fields option

2 participants