rikatee

9 ms·

31.
by rikatee 5y ago
yeah the impact varies. the sentry one seems pretty big: https://codereviewdoctor.medium.com/5-of-666-python-repos-ha... test did not work but did not fail either, imagine being that dev maintaining the code that the test p
32.
by rikatee 5y ago
yeah something like sonarqube or https://codereview.doctor (if you use GitHub)
33.
5% of 666 Python repos had comma typo bugs (inc V8, TensorFlow and PyTorch)(codereviewdoctor.medium.com)
360 points by rikatee 5y ago|327 comments
34.
by rikatee 5y ago
http://codereview.doctor/ It's static analysis SaaS that detects bugs and code smells and offers the fix right inside the GitHub PR, so no need to context switch - just click commit and continue with your day
35.
Show HN: I made a bot that suggests fixes for Python bugs right inside the PR(codereview.doctor)
9 points by rikatee 5y ago|1 comments
36.
by rikatee 5y ago
It's a code review copilot for GitHub that suggests fixes to bugs it finds e.g, if you missed a comma in a list, accidentally committed a breakpoint, used deprecated features, using "==" when you should use "is" and
37.
by rikatee 5y ago
We use python's AST to power https://pypi.org/project/django-doctor/ - it's a Django linter that suggests the fix and makes the change it suggests if you approve the change. Eventually we will replace AS
38.
Show HN: Django Code Improvement Suggestion Engine(pypi.org)
1 points by rikatee 5y ago|1 comments
39.
by rikatee 5y ago
6 months ago I released Django Doctor to review github pull requests. Now it can be installed as a command line tool via `pip install django-doctor`. So now instead of reviewing deltas in GitHub PRs it runs locally and analyses your entire
40.
by rikatee 6y ago
Django release notes are great https://docs.djangoproject.com/en/3.1/releases/ but I got frustrated that when upgrading multiple versions it was not easy to get a summary of relevant changes as it's one
41.
Show HN: Django changelog comparison tool – see what changed between releases(django.doctor)
4 points by rikatee 6y ago|2 comments
42.
Django security challenge: can you fix all the security flaws?(django.doctor)
2 points by rikatee 6y ago|0 comments
43.
Django refactor challenge: can you fix all the ORM anti patterns?(django.doctor)
2 points by rikatee 6y ago|0 comments
44.
by rikatee 6y ago
Now we can remove baked in tech debt from Django codebase, not just detect it during code review (previously https://django.doctor could only suggest improvements to github PRs)
45.
by rikatee 6y ago
glad you liked it :)
46.
by rikatee 6y ago
If you submit the same answer twice then it adds them twice. Future version of the quiz will close that bug :)
47.
by rikatee 6y ago
for that you need a human unfortunately ;)
48.
by rikatee 6y ago
on the roadmap is config file but for now devs can just ignore the advice :)
49.
by rikatee 6y ago
This covers it quite nicely https://django.doctor/advice/C2006
50.
by rikatee 6y ago
wow well done!
51.
by rikatee 6y ago
In the real world the GitHub PR bot gives the dev the choice of committing the advice or not. I appreciate that does not reflect perfectly in the game :) > I also don't see anything wrong with explicit default arguments. Very much y
52.
by rikatee 6y ago
I think not because a think the bot cannot know is "why". An anti-pattern may be done for solid business reasons. It's Django Doctor's job to say "x could be y" and it's the dev's job to approve/
53.
by rikatee 6y ago
You're right - that's why the GitHub bot just gives "food for thought" advice and does not block the merge. It's "have you considered" rather than "you must do this". The dev has the choice of if
54.
by rikatee 6y ago
Well done. Shows either you're a very intuitive dev, or Django is a very intuitive tool. Maybe both. These problems I think are more to do with inertia and not having best practices in mind. It's easy to do CharField and not thing
55.
by rikatee 6y ago
I think it's a good thing. It indicates: - Django is big (because it solves a lot of problems) - there exists a learning curve Sometimes people miss best practices when the interplay between those two points occur. Fortunately bots exi
56.
by rikatee 6y ago
Thanks! and blimey well done that's better than I did first go around
57.
Django refactoring game – can you fix all the Models anti-patterns?(django.doctor)
80 points by rikatee 6y ago|53 comments
58.
by rikatee 6y ago
I made this as a tech demo for what https://django.doctor is capable of doing in Pull Requests when it reviews your code. Django Doctor is a code improvement bot you can add to GitHub to review your PRs
59.
Spot urls.py bugs lurking in plain sight(djangodoctor.medium.com)
1 points by rikatee 6y ago|0 comments
60.
by rikatee 6y ago
We now support private repos :)

More ›