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

docs: update ranking information #2749

Closed
wants to merge 4 commits into from
Closed

docs: update ranking information #2749

wants to merge 4 commits into from

Conversation

rickstaa
Copy link
Collaborator

Update the documentation to the new ranking mechanism #1186. @francois-rozet please check if the documentation is correct :).

@vercel
Copy link

vercel bot commented May 26, 2023

The latest updates on your projects. Learn more about Vercel for Git ↗︎

Name Status Preview Updated (UTC)
github-readme-stats ✅ Ready (Inspect) Visit Preview May 30, 2023 8:02am

@rickstaa rickstaa requested a review from Zo-Bro-23 May 26, 2023 14:21
@github-actions github-actions bot added the documentation Improvements or additions to documentation. label May 26, 2023
@rickstaa rickstaa requested a review from anuraghazra May 26, 2023 14:21
@codecov
Copy link

codecov bot commented May 26, 2023

Codecov Report

Patch coverage has no change and project coverage change: +0.03 🎉

Comparison is base (c96e84a) 97.52% compared to head (46bed01) 97.55%.

Additional details and impacted files
@@            Coverage Diff             @@
##           master    #2749      +/-   ##
==========================================
+ Coverage   97.52%   97.55%   +0.03%     
==========================================
  Files          24       24              
  Lines        4683     4742      +59     
  Branches      430      493      +63     
==========================================
+ Hits         4567     4626      +59     
  Misses        115      115              
  Partials        1        1              

see 2 files with indirect coverage changes

☔ View full report in Codecov by Sentry.
📢 Do you have feedback about the report comment? Let us know in this issue.

readme.md Outdated Show resolved Hide resolved
Co-authored-by: François Rozet <francois.rozet@outlook.com>
@francois-rozet
Copy link
Collaborator

@rickstaa I have submitted a PR #2762 that further modifies the ranking system. We should merge it first (after review) before updating the doc.

Copy link
Collaborator

@qwerty541 qwerty541 left a comment

Choose a reason for hiding this comment

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

This changes are already inside master branch? Right?

@rickstaa
Copy link
Collaborator Author

This changes are already inside master branch? Right?

Correct 👍🏻!

@rickstaa rickstaa closed this Jun 15, 2023
@rickstaa rickstaa deleted the update_rank_doc branch June 15, 2023 07:25
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
documentation Improvements or additions to documentation.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants