A Philosophy of Code Review
benpastel.com4 pointsby _pastel1 comments
select *
from report_card
where dataset_query ilike '%' || {{query}} || '%'
(You can also join in metadata like the author, when it was last ran, etc.) acks_late=True, task_reject_on_worker_lost=True
but not in cases like a worker hanging indefinitely without being explicitly killed. - www.reddit.com/r/dalle2/comments/uzosy1/the_rest_of_mona_lisa
- www.reddit.com/r/dalle2/comments/vstuns/super_mario_getting_his_citizenship_at_ellis
And others are hilarious: - www.reddit.com/r/dalle2/comments/v0pjfr/a_photograph_of_a_street_sign_that_warns_drivers
- www.reddit.com/r/dalle2/comments/wbbkbb/healthy_food_at_mcdonalds
- www.reddit.com/r/dalle2/comments/wlfpax/the_elements_of_fire_water_earth_and_air_digital
Even within the same project, for a given PR, there are some parts of the codebase I want to modify freely and some that I want fixed to reduce the diff and testing scope.
I try to explain up-front to the agent how aggressively they can modify the existing code and which parts, but I've had mixed success; usually they bias towards a minimal diff even if that means duplication or abusing some abstractions. If anyone has had better success, I'd love to hear your approach.