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

Skip to content

Conversation

@stscieisenhamer
Copy link
Collaborator

@stscieisenhamer stscieisenhamer commented Dec 12, 2022

Resolves user incident report INC0184702

This PR addresses yet another typo in the naming of the CRDS_PATH environment variable.

Developer cannot be trusted to write documentation...

Checklist for maintainers

  • added entry in CHANGES.rst within the relevant release section
  • updated or added relevant tests
  • updated relevant documentation
  • added relevant milestone
  • added relevant label(s)
  • ran regression tests, post a link to the Jenkins job below.
  • Make sure the JIRA ticket is resolved properly

@codecov
Copy link

codecov bot commented Dec 12, 2022

Codecov Report

Base: 79.89% // Head: 79.90% // Increases project coverage by +0.00% 🎉

Coverage data is based on head (46c37b8) compared to base (9fe4938).
Patch has no changes to coverable lines.

Additional details and impacted files
@@           Coverage Diff           @@
##           master    #7392   +/-   ##
=======================================
  Coverage   79.89%   79.90%           
=======================================
  Files         413      413           
  Lines       37975    37914   -61     
=======================================
- Hits        30341    30296   -45     
+ Misses       7634     7618   -16     
Flag Coverage Δ *Carryforward flag
nightly 79.88% <ø> (ø) Carriedforward from 9fe4938
unit 51.84% <ø> (-0.05%) ⬇️

*This pull request uses carry forward flags. Click here to find out more.

Impacted Files Coverage Δ
jwst/associations/lib/utilities.py 85.71% <0.00%> (-2.39%) ⬇️
jwst/ipc/x_irs2.py 10.46% <0.00%> (-1.03%) ⬇️
jwst/saturation/x_irs2.py 44.18% <0.00%> (-0.65%) ⬇️
jwst/associations/registry.py 99.40% <0.00%> (-0.60%) ⬇️
jwst/tweakreg/astrometric_utils.py 88.52% <0.00%> (-0.37%) ⬇️
jwst/associations/association.py 91.35% <0.00%> (-0.32%) ⬇️
jwst/associations/lib/process_list.py 84.29% <0.00%> (-0.26%) ⬇️
jwst/lib/signal_slot.py 81.39% <0.00%> (-0.22%) ⬇️
jwst/associations/lib/product_utils.py 94.73% <0.00%> (-0.10%) ⬇️
jwst/extract_1d/extract.py 79.59% <0.00%> (-0.06%) ⬇️
... and 13 more

Help us with your feedback. Take ten seconds to tell us how you rate us. Have a feature suggestion? Share it here.

☔ View full report at Codecov.
📢 Do you have feedback about the report comment? Let us know in this issue.

@stscieisenhamer stscieisenhamer merged commit 62927ea into master Dec 12, 2022
zacharyburnett pushed a commit to zacharyburnett/jwst that referenced this pull request Dec 12, 2022
spacetelescope#7392)

Changes
* Update deprecation notice to name the CRDS_PATH variable appropriately
* Update CHANGES.rst

(cherry picked from commit 62927ea)
zacharyburnett pushed a commit that referenced this pull request Dec 12, 2022
#7392)

Changes
* Update deprecation notice to name the CRDS_PATH variable appropriately
* Update CHANGES.rst

(cherry picked from commit 62927ea)
zacharyburnett pushed a commit to zacharyburnett/jwst that referenced this pull request Dec 12, 2022
spacetelescope#7392)

Changes
* Update deprecation notice to name the CRDS_PATH variable appropriately
* Update CHANGES.rst

(cherry picked from commit 62927ea)
zacharyburnett added a commit that referenced this pull request Dec 12, 2022
)

* release procedures

* JP-3003 Clarify how to manage a local CRDS cache (#7350)

Changes
* Remove any mention of "rm" from the deprecation notice
* shorten the example path in the introduction
* update changelog
* Add information about content and size requirements
* minor word change on the deprecation instructions

(cherry picked from commit 1b8d288)

* Update deprecation notice to name the CRDS_PATH variable appropriately (#7392)

Changes
* Update deprecation notice to name the CRDS_PATH variable appropriately
* Update CHANGES.rst

(cherry picked from commit 62927ea)

Co-authored-by: Jonathan Eisenhamer <[email protected]>
@braingram braingram deleted the pubdep_fixcrdspath branch October 3, 2024 09:36
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants