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

Fixes related to sample data updates #523

Draft
wants to merge 20 commits into
base: main
Choose a base branch
from

Conversation

Cadair
Copy link
Member

@Cadair Cadair commented Feb 8, 2025

Closes #522
Closes #527

@Cadair Cadair added the Run Publish Run publish CI on this PR label Feb 8, 2025
@Cadair
Copy link
Member Author

Cadair commented Feb 8, 2025

pre-commit.ci autofix

pyproject.toml Outdated Show resolved Hide resolved
Copy link

codspeed-hq bot commented Feb 10, 2025

CodSpeed Performance Report

Merging #523 will not alter performance

Comparing Cadair:post-1.10-fixes (554ea87) with main (75421e8)

Summary

✅ 11 untouched benchmarks

@SolarDrew

This comment was marked as resolved.

tox.ini Show resolved Hide resolved
pytest.ini Outdated
@@ -70,3 +70,5 @@ filterwarnings =
ignore:pkg_resources is deprecated as an API.*:DeprecationWarning
ignore:Deprecated call to .*pkg_resources\.declare_namespace.*mpl_toolkits.*:DeprecationWarning
ignore:Extension .*sunpy-1\.0\.0:asdf.exceptions.AsdfManifestURIMismatchWarning
# This warning should go away in asdf-astropy 0.7.1 and we can stop being ignored then
ignore::asdf.exceptions.AsdfPackageVersionWarning
Copy link
Contributor

Choose a reason for hiding this comment

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

FYI: 0.7.1 with the fix is up on pypi. https://pypi.org/project/asdf-astropy/0.7.1/

changelog/523.bugfix.rst Outdated Show resolved Hide resolved
pytest.ini Outdated Show resolved Hide resolved
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Run Publish Run publish CI on this PR
Projects
None yet
3 participants