-
Notifications
You must be signed in to change notification settings - Fork 284
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
Remove freepik.com link due to linkcheck breakage #6261
Conversation
Codecov ReportAll modified and coverable lines are covered by tests ✅
Additional details and impacted files@@ Coverage Diff @@
## main #6261 +/- ##
=======================================
Coverage 89.83% 89.83%
=======================================
Files 88 88
Lines 23315 23315
Branches 4338 4338
=======================================
Hits 20945 20945
Misses 1644 1644
Partials 726 726 ☔ View full report in Codecov by Sentry. |
@ukmo-ccbunney thanks for volunteering your time, this should be a quick one! I decided the link didn't deliver any value so it wasn't worth pursuing and just removed it instead. |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Odd how the link ignore didn't work, that was going to be my suggestion until I saw you already tried it.
I agree that we can just remove it, especially as it is blocking other work.
Thanks @ukmo-ccbunney! |
…ask_slicing_bug * origin/pin_dask_slicing_bug: `date_microseconds` FUTURE flag (SciTools#6260) [pre-commit.ci] pre-commit autoupdate (SciTools#6259) Remove freepik.com link due to linkcheck breakage (SciTools#6261)
🚀 Pull Request
Description
Consult Iris pull request check list
Add any of the below labels to trigger actions on this PR: