Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Jump to bottom

replace usage of tox with direct pytest commands #161

Merged
merged 12 commits into from
Aug 2, 2024
Merged

replace usage of tox with direct pytest commands #161

merged 12 commits into from
Aug 2, 2024

Conversation

zacharyburnett
Copy link
Collaborator

No description provided.

@zacharyburnett zacharyburnett self-assigned this Jul 19, 2024
@zacharyburnett zacharyburnett changed the title run pytest directly (tox does not use environment) replace usage of tox with direct pytest commands Jul 19, 2024
@zacharyburnett
Copy link
Collaborator Author

zacharyburnett commented Aug 1, 2024 •
edited
Loading

issues:

  • AstroDrizzle Version 3.7.0.1 encountered a problem! 
    File "/home/runner/micromamba/envs/stenv-Linux-py3.10/lib/python3.10/site-packages/stsci/tools/check_files.py", line 18, in checkFiles
  if isinstance(filelist[0], str):
IndexError: list index out of range
  • ModuleNotFoundError: Error importing 'from crds.jwst import locate' : No module named 'pkg_resources'
  • E synphot.exceptions.SynphotError: Vega spectrum is missing.
    • fixed by removing --remote-data 5fbbcf0

@zacharyburnett zacharyburnett force-pushed the tox/use_environment branch 2 times, most recently from 7a7a248 to af74fad Compare August 2, 2024 13:04
@zacharyburnett zacharyburnett marked this pull request as ready for review August 2, 2024 16:51
@zacharyburnett zacharyburnett requested a review from a team as a code owner August 2, 2024 16:51
@zacharyburnett zacharyburnett merged commit d1f36c2 into spacetelescope:main Aug 2, 2024
69 of 80 checks passed
@zacharyburnett zacharyburnett deleted the tox/use_environment branch August 2, 2024 16:51
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers
No reviews
Assignees

@zacharyburnett zacharyburnett

Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
1 participant
@zacharyburnett