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

feat: New Coverage engine - diff coverage toggle ALA-574 #1872

Merged

Conversation

nicklem
Copy link
Contributor

@nicklem nicklem commented Sep 19, 2023

Documents the new diff coverage toggle. It will be available for GitHub only for now.

👀 Live preview

  • GitHub
  • GitLab (no updates for now)
  • Bitbucket (no updates for now)

🚧 To do

  • Remove feature for GitLab and Bitbucket.
  • Review screenshots. They should already match the prod version.
  • Request validation
  • Apply feedback
  • Wait for feature release

@nicklem nicklem self-assigned this Sep 19, 2023
@github-actions
Copy link
Contributor

github-actions bot commented Sep 19, 2023

Overall readability score: 53.13 (🟢 +0.02)

File Readability
github-integration.md 40.78 (🟢 +3.66)
View detailed metrics

🟢 - Shows an increase in readability
🔴 - Shows a decrease in readability

File Readability FRE GF ARI CLI DCRS
github-integration.md 40.78 24.07 12.07 17 14.91 7.3
  🟢 +3.66 🔴 -0.1 🟢 +0.36 🟢 +0.7 🟢 +0.7 🟢 +0.33

Averages:

  Readability FRE GF ARI CLI DCRS
Average 53.13 42.04 10.95 12.7 12.73 7.88
  🟢 +0.02 🟢 +0 🟢 +0 🟢 +0 🟢 +0 🟢 +0
View metric targets
Metric Range Ideal score
Flesch Reading Ease 100 (very easy read) to 0 (extremely difficult read) 60
Gunning Fog 6 (very easy read) to 17 (extremely difficult read) 8 or less
Auto. Read. Index 6 (very easy read) to 14 (extremely difficult read) 8 or less
Coleman Liau Index 6 (very easy read) to 17 (extremely difficult read) 8 or less
Dale-Chall Readability 4.9 (very easy read) to 9.9 (extremely difficult read) 6.9 or less

@nicklem nicklem force-pushed the ALA-574-doc-docs-new-coverage-engine-status-checks-toggle branch from aff752a to 73dbc8d Compare September 19, 2023 10:59
@github-actions github-actions bot temporarily deployed to Netlify September 19, 2023 11:01 Inactive
@github-actions github-actions bot temporarily deployed to Netlify September 20, 2023 08:55 Inactive
@nicklem nicklem changed the title feat: Document the new diff coverage toggle ALA-574 feat: New Coverage engine - diff coverage toggle ALA-574 Sep 20, 2023
@nicklem nicklem added the don't merge Don't merge this pull request yet label Sep 20, 2023
@github-actions github-actions bot temporarily deployed to Netlify September 21, 2023 09:03 Inactive
@nicklem nicklem force-pushed the ALA-574-doc-docs-new-coverage-engine-status-checks-toggle branch from 275d3d9 to 1ed8a59 Compare September 22, 2023 09:51
@github-actions github-actions bot temporarily deployed to Netlify September 22, 2023 09:53 Inactive
@github-actions github-actions bot temporarily deployed to Netlify September 29, 2023 15:38 Inactive
@claudiacarpinteiro claudiacarpinteiro self-assigned this Sep 29, 2023
@github-actions github-actions bot temporarily deployed to Netlify September 29, 2023 16:15 Inactive
@github-actions github-actions bot temporarily deployed to Netlify September 29, 2023 16:44 Inactive
@github-actions github-actions bot temporarily deployed to Netlify September 29, 2023 17:02 Inactive
@github-actions github-actions bot temporarily deployed to Netlify September 29, 2023 17:44 Inactive
@github-actions github-actions bot temporarily deployed to Netlify September 29, 2023 18:06 Inactive
@claudiacarpinteiro claudiacarpinteiro marked this pull request as ready for review September 29, 2023 18:08
@claudiacarpinteiro claudiacarpinteiro requested a review from a team as a code owner September 29, 2023 18:08
@claudiacarpinteiro claudiacarpinteiro requested a review from a team September 29, 2023 18:09
Copy link
Member

@machadoit machadoit left a comment

Choose a reason for hiding this comment

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

LGTM!

@machadoit machadoit removed the don't merge Don't merge this pull request yet label Oct 4, 2023
@machadoit machadoit merged commit b8ae926 into master Oct 4, 2023
7 of 8 checks passed
@machadoit machadoit deleted the ALA-574-doc-docs-new-coverage-engine-status-checks-toggle branch October 4, 2023 11:49
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.

3 participants