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.

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

temporarily remove jdaviz from devdeps #102

Merged
merged 1 commit into from
Mar 28, 2024

Conversation

kecnry
Copy link
Member

@kecnry kecnry commented Mar 28, 2024

We're currently pinning a minor version of jdaviz and handling updates manually, so want to test that version of jdaviz with other devdeps (so a break caused by jdaviz does not prevent us from testing the other devdeps). Updating the pin to the next minor version of jdaviz is currently handled by PRs that bump the version and apply any necessary changes (i.e. #68) which will then ensure tests pass before bumping the minor version of jdaviz.

If we ever stop tagging a specific minor version, dev tests for jdaviz should be re-introduced (as mentioned in the added comment in pyproject.toml)

* we're currently pinning a minor version of jdaviz and handling updates manually, so want to test that version of jdaviz with other devdeps
@kecnry kecnry marked this pull request as ready for review March 28, 2024 18:13
@kecnry kecnry requested a review from rosteen March 28, 2024 18:13
@rosteen
Copy link
Contributor

rosteen commented Mar 28, 2024

Seems like a good plan for now.

@kecnry kecnry merged commit 8397989 into spacetelescope:main Mar 28, 2024
9 checks passed
@kecnry kecnry deleted the devdeps-no-jdaviz branch March 28, 2024 18:38
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