Thanks to visit codestin.com
Credit goes to github.com

Skip to content

Conversation

@alphasentaurii
Copy link
Collaborator

@alphasentaurii alphasentaurii commented Aug 20, 2025

Resolves CCD-1635

This PR addresses ...
Running the crds_s3_get script without first installing the aws-related optional dependencies (pip install crds['aws']) produces a traceback that does not make clear what actually is causing the error. The script has been updated to first check to make sure the necessary libraries are installed and produces a helpful explanation if either awscli or boto3 are missing when the error is caught.

Tasks

  • update or add relevant tests
  • update relevant docstrings and / or docs/ page
  • Does this PR change any API used downstream? (if not, label with no-changelog-entry-needed)
    • write news fragment(s) in changes/: echo "changed something" > changes/<PR#>.<changetype>.rst (see below for change types)
news fragment change types...
  • changes/<PR#>.hst.rst: HST reference files
  • changes/<PR#>.jwst.rst: JWST reference files
  • changes/<PR#>.roman.rst: Roman reference files
  • changes/<PR#>.doc.rst: documentation change
  • changes/<PR#>.testing.rst: change to tests or test automation
  • changes/<PR#>.general.rst: infrastructure or miscellaneous change

@alphasentaurii alphasentaurii requested a review from a team as a code owner August 20, 2025 19:14
@alphasentaurii alphasentaurii merged commit 29a33cb into spacetelescope:master Aug 26, 2025
10 checks passed
alphasentaurii added a commit that referenced this pull request Aug 26, 2025
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.

1 participant