Skip to content
This repository was archived by the owner on Aug 28, 2025. It is now read-only.

Conversation

Borda
Copy link
Member

@Borda Borda commented Jan 8, 2024

Before submitting

  • Was this discussed/approved via a Github issue? (no need for typos and docs improvements)
  • Did you make sure to update the docs?
  • Did you write any new necessary tests?

What does this PR do?

linking PDF is quite heavy and sometimes returning false check while we validate correct link

PR review

Anyone in the community is free to review the PR once the tests have passed.
If we didn't discuss your PR in Github issues there's a high chance it will not be merged.

Did you have fun?

Make sure you had fun coding 🙃

Copy link

codecov bot commented Jan 8, 2024

Codecov Report

All modified and coverable lines are covered by tests ✅

Project coverage is 70%. Comparing base (30b6397) to head (4c2be2f).

Additional details and impacted files
@@         Coverage Diff         @@
##           main   #305   +/-   ##
===================================
  Coverage    70%    70%           
===================================
  Files         2      2           
  Lines       403    403           
===================================
  Hits        284    284           
  Misses      119    119           

@Borda Borda force-pushed the main branch 2 times, most recently from d4acf6f to ac8f7ba Compare July 19, 2024 21:12
@Borda Borda merged commit cbbea35 into main Jul 20, 2024
@Borda Borda deleted the docs/link-arxiv-abs branch July 20, 2024 08:22
@Borda Borda added the enhancement New feature or request label Jul 23, 2024
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

1 participant