Skip to content

Comments

Links update#251

Merged
sanjanag merged 18 commits intocleanlab:mainfrom
sanjanag:links-update
Feb 13, 2024
Merged

Links update#251
sanjanag merged 18 commits intocleanlab:mainfrom
sanjanag:links-update

Conversation

@sanjanag
Copy link
Contributor

@sanjanag sanjanag commented Feb 12, 2024

Updates to documentation

  • changed http links to relative links
  • updated dev guide and readme with better instructions
  • update docs requirements
  • update html proofer

Preview: https://cleanvision--251.org.readthedocs.build/en/251/

@review-notebook-app
Copy link

Check out this pull request on  ReviewNB

See visual diffs & provide feedback on Jupyter Notebooks.


Powered by ReviewNB

@codecov
Copy link

codecov bot commented Feb 12, 2024

Codecov Report

All modified and coverable lines are covered by tests ✅

Comparison is base (972f060) 96.10% compared to head (cd85485) 96.10%.

Additional details and impacted files
@@           Coverage Diff           @@
##             main     #251   +/-   ##
=======================================
  Coverage   96.10%   96.10%           
=======================================
  Files          16       16           
  Lines        1001     1001           
  Branches      195      195           
=======================================
  Hits          962      962           
  Misses         19       19           
  Partials       20       20           

☔ View full report in Codecov by Sentry.
📢 Have feedback on the report? Share it here.

@sanjanag sanjanag marked this pull request as ready for review February 13, 2024 01:48
@sanjanag sanjanag requested a review from jwmueller February 13, 2024 07:54
Copy link
Member

@jwmueller jwmueller left a comment

Choose a reason for hiding this comment

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

lgtm

@sanjanag sanjanag merged commit 8d8ffaf into cleanlab:main Feb 13, 2024
@sanjanag sanjanag deleted the links-update branch February 13, 2024 17:13
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants