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: Add option to toggle coverage details in VS Code extension #2086

Open
wants to merge 12 commits into
base: master
Choose a base branch
from

Conversation

claudiacarpinteiro
Copy link
Contributor

@claudiacarpinteiro claudiacarpinteiro commented Apr 22, 2024

Updates VS Code extension documentation with the new option to toggle on/off the line-by-line coverage variation introduced by the pull request to each file (see codacy/codacy-vscode-extension#23).

👀 Live preview

https://docs-681-add-coverage-annotation--docs-codacy.netlify.app/getting-started/integrating-codacy-with-visual-studio-code/#status-tab

🚧 To do

  • Update content considering the new behavior
  • Update screenshot
  • Request validation
  • Apply feedback
  • Wait for feature release

@claudiacarpinteiro claudiacarpinteiro self-assigned this Apr 22, 2024
@claudiacarpinteiro claudiacarpinteiro added the don't merge Don't merge this pull request yet label Apr 22, 2024
Copy link
Contributor

github-actions bot commented Apr 22, 2024

Overall readability score: 53.83 (🔴 -0.01)

File Readability
integrating-codacy-with-visual-studio-code.md 67.84 (🔴 -1.28)
View detailed metrics

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

File Readability FRE GF ARI CLI DCRS
integrating-codacy-with-visual-studio-code.md 67.84 52.6 10.02 10.8 10.04 6.38
  🔴 -1.28 🔴 -0.5 🔴 -0.19 🔴 -0.3 🔴 -0.24 🔴 -0.03

Averages:

  Readability FRE GF ARI CLI DCRS
Average 53.83 42.54 10.89 12.45 12.56 7.91
  🔴 -0.01 🟢 +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

@github-actions github-actions bot temporarily deployed to Netlify April 22, 2024 16:24 Inactive
@github-actions github-actions bot temporarily deployed to Netlify April 23, 2024 08:58 Inactive
@github-actions github-actions bot temporarily deployed to Netlify April 26, 2024 09:55 Inactive
@github-actions github-actions bot temporarily deployed to Netlify April 30, 2024 15:33 Inactive
@github-actions github-actions bot temporarily deployed to Netlify April 30, 2024 16:11 Inactive
@github-actions github-actions bot temporarily deployed to Netlify April 30, 2024 16:36 Inactive
@github-actions github-actions bot temporarily deployed to Netlify April 30, 2024 16:39 Inactive
Copy link
Contributor

@nicklem nicklem left a comment

Choose a reason for hiding this comment

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

LGTM 👍 The screenshot does help in the end

@github-actions github-actions bot temporarily deployed to Netlify May 2, 2024 11:58 Inactive
@claudiacarpinteiro claudiacarpinteiro marked this pull request as ready for review May 2, 2024 11:58
@claudiacarpinteiro claudiacarpinteiro requested a review from a team as a code owner May 2, 2024 11:58
@github-actions github-actions bot temporarily deployed to Netlify May 7, 2024 11:30 Inactive
@github-actions github-actions bot temporarily deployed to Netlify May 8, 2024 10:56 Inactive
@github-actions github-actions bot temporarily deployed to Netlify May 9, 2024 14:18 Inactive
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
don't merge Don't merge this pull request yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

2 participants