Related Posts
Everyone: Excel is a smart app
Excel:

Anyone familiar with SmartHop?
Additional Posts in Software Engineering
Where/ how can I learn Azure?
New to Fishbowl?
Download the Fishbowl app to
unlock all discussions on Fishbowl.
unlock all discussions on Fishbowl.





A 'You break it, you fix it' policy would take care of it pretty quick
lol this is right
Formalize feedback into a process. Either a direct one (lint rules/test rules/etc) or process rules. (Checklist for signoff)
If it leads to a problem/incident justify the change in process with those docs.
If it's just nits/nuance that doesn't lead to large problems at best you can document updates in a backlog or something.
What about when process is ignored?
Chief
I’d bring it up in a retro or bring it up to your project manager
Depending on the issue it's often better to enforce quality by design and automated checks. Your comments maybe great and even if they took action on the feed back you can't possibly catch every issues or review every PR.
Can the design/architecture be improved so that the anti-pattern is less likely? Can style checks or linters be used? to enforce project standards?
You can implement plugins that can help. For instance if it’s Java apps there’s some maven plugins that you can set a required unit test line coverage. If they don’t meet the coverage then the build fails
Is SAST in placeAnd part of definition of done to have certain number or grading of code smells?
I've been there, and the only way I found was to leave.