Skip to content

PEP 621 + src/ layout#41

Merged
stscirij merged 6 commits into
spacetelescope:masterfrom
jhunkeler:pep-621
Feb 11, 2026
Merged

PEP 621 + src/ layout#41
stscirij merged 6 commits into
spacetelescope:masterfrom
jhunkeler:pep-621

Conversation

@jhunkeler
Copy link
Copy Markdown
Contributor

@jhunkeler jhunkeler commented Feb 10, 2026

This PR does the following:

  • The project directory costools was moved to src/costools
  • setup.py has been replaced by pyproject.toml. The script entry points have been carried over
  • The defunct azure-pipelines.yml configuration was removed
  • The pypi upload workflow has been updated to use OpenAstronomy's reusable workflow. The wheel/sdist build will kick off for each PR and push to master. Uploads to pypi only occur when a release object is published
  • Dependabot has been added

@jhunkeler jhunkeler requested a review from a team as a code owner February 10, 2026 20:44
@codecov
Copy link
Copy Markdown

codecov Bot commented Feb 10, 2026

Codecov Report

✅ All modified and coverable lines are covered by tests.
✅ Project coverage is 11.12%. Comparing base (58422d3) to head (a2ed935).
⚠️ Report is 2 commits behind head on master.

Additional details and impacted files
@@            Coverage Diff             @@
##           master      #41      +/-   ##
==========================================
+ Coverage   10.94%   11.12%   +0.18%     
==========================================
  Files           7        8       +1     
  Lines         987      989       +2     
  Branches      195      195              
==========================================
+ Hits          108      110       +2     
  Misses        879      879              

☔ View full report in Codecov by Sentry.
📢 Have feedback on the report? Share it here.

🚀 New features to boost your workflow:
  • ❄️ Test Analytics: Detect flaky tests, report on failures, and find test suite problems.

* Replace with build and test workflow status
Copy link
Copy Markdown
Contributor

@stscirij stscirij left a comment

Choose a reason for hiding this comment

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

I think the only reason this needs to be in the build is because of the add_cos_s_region script/module. I tested installing from this and running the script and it seems to work OK.

@stscirij stscirij merged commit 5bf5eb6 into spacetelescope:master Feb 11, 2026
13 checks passed
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.

2 participants