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

Minor changes to remote cronstests and refactoring workflow cancel #2406

Merged
merged 3 commits into from
May 17, 2022

Conversation

bsipocz
Copy link
Member

@bsipocz bsipocz commented May 17, 2022

The job is very badly times out, probably need to address #2405 first as I cannot reproduce the issue locally.

@codecov
Copy link

codecov bot commented May 17, 2022

Codecov Report

Merging #2406 (1ec072a) into main (b2bd214) will not change coverage.
The diff coverage is n/a.

❗ Current head 1ec072a differs from pull request most recent head 013bf0d. Consider uploading reports for the commit 013bf0d to get more accurate results

@@           Coverage Diff           @@
##             main    #2406   +/-   ##
=======================================
  Coverage   62.96%   62.96%           
=======================================
  Files         133      133           
  Lines       17287    17287           
=======================================
  Hits        10885    10885           
  Misses       6402     6402           

📣 Codecov can now indicate which changes are the most critical in Pull Requests. Learn more

@bsipocz bsipocz changed the title Minor changes to remote cronstests Minor changes to remote cronstests and refactoring workflow cancel May 17, 2022
@bsipocz
Copy link
Member Author

bsipocz commented May 17, 2022

Double checked with a force push, this indeed works super nicely.

@bsipocz bsipocz merged commit ffb6c52 into astropy:main May 17, 2022
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

1 participant