code-review-emoji-guide
Qodana
code-review-emoji-guide | Qodana | |
---|---|---|
8 | 3 | |
431 | 389 | |
- | 0.0% | |
0.0 | 9.3 | |
about 2 years ago | 8 days ago | |
- | Apache License 2.0 |
Stars - the number of stars that a project has on GitHub. Growth - month over month growth in stars.
Activity is a relative number indicating how actively a project is being developed. Recent commits have higher weight than older ones.
For example, an activity of 9.0 indicates that a project is amongst the top 10% of the most actively developed projects that we are tracking.
code-review-emoji-guide
-
Do you guys give/get positive feedback in code reviews?
I give it here and there, depends how much time I have. I recommend using an emoji code if you don't already as it allows you to easily give positive feedback or mark areas for tech debt etc: https://github.com/erikthedeveloper/code-review-emoji-guide
- Difficulties in mentoring junior and mid level developers
-
How do you prevent PRs from getting stuck in your teams?
self reviewing your PRs definitely makes it easier for reviewers to ramp up fast. i found setting review guidelines (https://github.com/erikthedeveloper/code-review-emoji-guide for example) helps give good actionable reviews (especially for self reviews)
-
Code review comments / slack messages often ignored
setting standards so that you're all on the same page wrt tone and circulating that internally could help. There's a code review emoji guide you could use (it's the first result in google for "code review emoji guide") or you could write your own. This can help make it more likely that no one is misinterpreting you (although it's impossible to make certain of this).
-
Do people think your PR comments sound arrogant or judgmental?
I like the ⛏ "pickaxe" emoji for these ('pickaxe' as in 'pick' as in 'nitpick').
-
Bluntly: how do I know if I'm a shit programmer
Try suggesting an emoji code review process: https://github.com/erikthedeveloper/code-review-emoji-guide
-
How do you deal with snarky PR comments?
Nice. I’ve used this emoji system and it works quite well too.
Qodana
-
Unstable activity happening with Qodana and .NET
It’s easy enough to migrate to the newer EAP version of the .NET linter which supports the latest version of .NET 8. You can view instructions on how to do this here.
-
Two Wishes for Dev Tooling
You'll want to keep an eye on Qodana[0] which when combined with ReviewDog[1] (and some glue script since Qodana has its own snowflake output JSON that RD doesn't read natively; engineering!) can offer suggested changes on MR/PR platforms which support such a thing [2]
I have the first two working together but not the last part yet, and Qodana is for sure a moving target but is what I've been praying for them to do for years now
0: https://github.com/JetBrains/Qodana/blob/2021.2/topics/getti...
1: https://github.com/reviewdog/reviewdog
2: https://docs.gitlab.com/ee/user/project/merge_requests/revie...
-
Using PhpStorm Php 8 attributes
It is possible to run PhpStorm on CI with Qodana https://github.com/jetbrains/qodana
What are some alternatives?
all-about-code-review - This is a curated list of resources about code reviews: articles, tools, trainings...
sonar-php - :elephant: SonarPHP: PHP static analyzer for SonarQube & SonarLint
hantsy - it's me
reviewdog - 🐶 Automated code review tool integrated with any code analysis tools regardless of programming language
codewars.com - Issue tracker for Codewars
Psalm - A static analysis tool for finding errors in PHP applications
editorconfig-vim - EditorConfig plugin for Vim
drone - Gitness is an Open Source developer platform with Source Control management, Continuous Integration and Continuous Delivery. [Moved to: https://github.com/harness/gitness]
code-review-checklist - This code review checklist helps you be a more effective and efficient code reviewer.
phpstorm-attributes - PhpStorm specific attributes
SpookyGhost-artifacts - A repository to collect SpookyGhost artifacts generated by continuous integration
awesome-ci - List of Continuous Integration services