Skip to content

Clean up hstcal recipe#42

Merged
pllim merged 2 commits into
conda-forge:mainfrom
pllim:fix-meta
Apr 29, 2026
Merged

Clean up hstcal recipe#42
pllim merged 2 commits into
conda-forge:mainfrom
pllim:fix-meta

Conversation

@pllim

@pllim pllim commented Apr 29, 2026

Copy link
Copy Markdown
Contributor

Checklist

  • Used a personal fork of the feedstock to propose changes
  • Bumped the build number (if the version is unchanged)
  • Reset the build number to 0 (if the version changed)
  • Re-rendered with the latest conda-smithy (Use the phrase @conda-forge-admin, please rerender in a comment in this PR for automated rerendering)
  • Ensured the license file is being packaged.

@conda-forge-admin

Copy link
Copy Markdown
Contributor

Hi! This is the friendly automated conda-forge-linting service.

I just wanted to let you know that I linted all conda-recipes in your PR (recipe/meta.yaml) and found it was in an excellent condition.

Comment thread recipe/meta.yaml
- {{ compiler('fortran') }}
- cmake
- make
- python # [build_platform != target_platform]

Copy link
Copy Markdown
Contributor Author

Choose a reason for hiding this comment

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

This is a pure C package now after we removed waf.

Comment thread recipe/meta.yaml
test:
commands:
- calacs.e --gitinfo
- calacs.e --version

Copy link
Copy Markdown
Contributor Author

Choose a reason for hiding this comment

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

Example output:

+ calacs.e --gitinfo
git tag: 77e4a09-dirty
git branch: HEAD
HEAD @: 77e4a099902f16168fccaf33ad9b4675ff81ea1d
+ calacs.e --version
10.4.1 (08-Aug-2025)
+ calwf3.e --version
3.7.3
+ cs0.e --version
3.5.0

Comment thread recipe/meta.yaml
license_file: LICENSE.txt
summary: Calibration for HST/WFC3, HST/ACS, and HST/STIS
doc_url: https://hstcal.readthedocs.io/
doc_url: https://github.com/spacetelescope/hstcal/blob/main/README.md

Copy link
Copy Markdown
Contributor Author

Choose a reason for hiding this comment

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

RTD content was never fully fleshed out. It just points back to the README now anyway.

Comment thread recipe/meta.yaml
recipe-maintainers:
- jhunkeler
- pllim
- nden

Copy link
Copy Markdown
Contributor Author

Choose a reason for hiding this comment

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

Nadia retired and her contract no longer includes hstcal.

@jhunkeler jhunkeler left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

LGTM

@pllim pllim merged commit 97040e9 into conda-forge:main Apr 29, 2026
6 checks passed
@pllim pllim deleted the fix-meta branch April 29, 2026 14:08
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.

3 participants