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

Merged
merged 13 commits into from
Jun 5, 2024

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: 54.04 (🔴 -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 54.04 43.08 10.8 12.46 12.57 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
@github-actions github-actions bot temporarily deployed to Netlify June 5, 2024 09:12 Inactive
@claudiacarpinteiro claudiacarpinteiro removed the don't merge Don't merge this pull request yet label Jun 5, 2024
@claudiacarpinteiro claudiacarpinteiro merged commit 2b44864 into master Jun 5, 2024
6 of 7 checks passed
@claudiacarpinteiro claudiacarpinteiro deleted the DOCS-681-add-coverage-annotation branch June 5, 2024 09:19
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.

None yet

3 participants