Remove mention of STAC_URL in Client.open docstring #317
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
Related Issue(s):
Description:
Quick patch that updates the docstring of
pystac_client.Client.open
to remove mentioning theSTAC_URL
environment variable, since it doesn't work anymore.Actually got a bit confused because the v0.2.0 changelog at https://github.com/stac-utils/pystac-client/blob/v0.5.0/CHANGELOG.md#v020---2021-08-04 says the fallback
STAC_URL
was added in #48 (5 May 2021), but the option was removed afterwards in #81 (2 Aug 2021) 😅Discovered this when writing unit tests at weiji14/zen3geo#59 and was wondering why I kept getting
AttributeError: 'NoneType' object has no attribute 'lower'
when usingurl=None
with theSTAC_URL
environment variable set. This was using pystac-client v0.4.0, but I think it's an issue from v0.2.0 to v0.5.0.PR Checklist: