-
Notifications
You must be signed in to change notification settings - Fork 41
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: VS Code plugin open PR list ALA-697 #1936
Conversation
Overall readability score: 54.33 (🟢 +0.01)
View detailed metrics🟢 - Shows an increase in readability
Averages:
View metric targets
|
181e68a
to
434010f
Compare
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Great work, @nicklem! 💪
Initially, I found it a bit weird to have an overview of the interface before the installation and remaining procedures for the user experience flow, but then I understood that we don't have really a use case for the new "tabs", except going there to see the information, right? 😉
So, I agree that this is the most logical way to structure the content we have at this point. 👌
I left some comments for your validation. 🤓
docs/getting-started/integrating-codacy-with-visual-studio-code.md
Outdated
Show resolved
Hide resolved
docs/getting-started/integrating-codacy-with-visual-studio-code.md
Outdated
Show resolved
Hide resolved
docs/getting-started/integrating-codacy-with-visual-studio-code.md
Outdated
Show resolved
Hide resolved
docs/getting-started/integrating-codacy-with-visual-studio-code.md
Outdated
Show resolved
Hide resolved
docs/getting-started/integrating-codacy-with-visual-studio-code.md
Outdated
Show resolved
Hide resolved
docs/getting-started/integrating-codacy-with-visual-studio-code.md
Outdated
Show resolved
Hide resolved
docs/getting-started/integrating-codacy-with-visual-studio-code.md
Outdated
Show resolved
Hide resolved
docs/getting-started/integrating-codacy-with-visual-studio-code.md
Outdated
Show resolved
Hide resolved
docs/getting-started/integrating-codacy-with-visual-studio-code.md
Outdated
Show resolved
Hide resolved
docs/getting-started/integrating-codacy-with-visual-studio-code.md
Outdated
Show resolved
Hide resolved
fa4c499
to
7036335
Compare
Co-authored-by: Cláudia Carpinteiro <[email protected]>
docs/getting-started/integrating-codacy-with-visual-studio-code.md
Outdated
Show resolved
Hide resolved
docs/getting-started/integrating-codacy-with-visual-studio-code.md
Outdated
Show resolved
Hide resolved
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
LGTM! 🚀
I left a small detail that you can follow up later on. 😉
…pr-list' into ALA-697-doc-vs-code-plugin-open-pr-list
Updates the VS Code extension documentation to cover the open PR list tab and other related updates.
👀 Live preview
https://ala-697-doc-vs-code-plugin-open-pr--docs-codacy.netlify.app/getting-started/integrating-codacy-with-visual-studio-code/
🚧 To do