Skip to content

Unskip jsoc tests#8120

Merged
nabobalis merged 1 commit into
sunpy:mainfrom
nabobalis:jsoc
Mar 6, 2025
Merged

Unskip jsoc tests#8120
nabobalis merged 1 commit into
sunpy:mainfrom
nabobalis:jsoc

Conversation

@nabobalis
Copy link
Copy Markdown
Member

@nabobalis nabobalis commented Mar 6, 2025

Need to see if the CI is happy.

There are some interment VSO/JSOC issues over the SOAP.

@nabobalis nabobalis added No Changelog Entry Needed Skip all changelog checks. Minor Change PR only needs one approval to merge. backport 6.0 labels Mar 6, 2025
Comment thread sunpy/net/tests/test_fido.py Outdated
@nabobalis nabobalis marked this pull request as ready for review March 6, 2025 22:02
@nabobalis nabobalis requested a review from a team as a code owner March 6, 2025 22:02
@nabobalis nabobalis merged commit 5b93f4e into sunpy:main Mar 6, 2025
@nabobalis nabobalis deleted the jsoc branch March 6, 2025 22:41
meeseeksmachine pushed a commit to meeseeksmachine/sunpy that referenced this pull request Mar 6, 2025
meeseeksmachine pushed a commit to meeseeksmachine/sunpy that referenced this pull request Mar 6, 2025
nabobalis added a commit that referenced this pull request Mar 6, 2025
…0-on-6.0

Backport PR #8120 on branch 6.0 (Unskip jsoc tests)
nabobalis added a commit that referenced this pull request Mar 6, 2025
…0-on-6.1

Backport PR #8120 on branch 6.1 (Unskip jsoc tests)
Comment thread output.txt
Comment on lines +1 to +3
ruff.....................................................................Passed
isort....................................................................Interrupted (^C): KeyboardInterrupt:
Check the log at /home/nabil/.cache/pre-commit/pre-commit.log
Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

Er...

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

@ayshih ayshih mentioned this pull request Mar 7, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Minor Change PR only needs one approval to merge. No Changelog Entry Needed Skip all changelog checks.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants